Skip to content

Feature/minsu#11

Open
ms-0o0 wants to merge 8 commits into
mainfrom
feature/minsu
Open

Feature/minsu#11
ms-0o0 wants to merge 8 commits into
mainfrom
feature/minsu

Conversation

@ms-0o0
Copy link
Copy Markdown
Contributor

@ms-0o0 ms-0o0 commented Dec 29, 2025

file 페이지 관련

  • api 로직 변경
  • 폴더 삭제 관한 내부적 파일도 삭제 처리
  • 폴더 다운로드 시에 zip 파일로 다운로드 되게끔 변경

@aaxx98
Copy link
Copy Markdown
Contributor

aaxx98 commented Dec 30, 2025

테스트 완료

  1. 폴더 생성
  2. 파일 업로드
  3. 파일 및 폴더 이동
  4. 파일 및 폴더명 변경
  5. 파일 및 폴더 다운로드
  6. 파일 및 폴더 삭제

수정 제안

  • .xlsx 파일 타입명이 길어서 디자인이 깨지고있습니다.
  • 목록 윗부분에 각 컬럼명이 표시되면 좋을거같습니다.
  • 팀별 저장 공간 용량을 확인하고 제한하는 기능이 필요할것같습니다.
스크린샷 2025-12-30 오후 6 40 23

ms-0o0 added 5 commits January 4, 2026 15:15
작성자 정보를 보내기 위한 정보 추가
작성자는 활동 member 명으로 보내게끔 로직 변경
WebConfig에서 corsConfig에 있는 Bean을 쓸 수 있게 하는 한 줄 코드 추가
어플리케이션 프라펄티에서 최대 파일 크기 10MB 지정
Controller에서 통신 관련 오류 해결
@ms-0o0
Copy link
Copy Markdown
Contributor Author

ms-0o0 commented Jan 7, 2026

현재 마무리 된 작업

  • Token 값을 통해 TeamId 및 MemberId 참조
  • Cors 정책 관련하여 코드 추가
CorsConfig (Bean 등록 방식)

Spring Security랑 같이 쓸 때 권장되는 방식

CorsConfigurationSource Bean으로 등록하면 Security 필터에서 CORS 적용 가능

Security 필터 전에 preflight 요청 처리 가능

WebConfig (WebMvcConfigurer 방식)

단순 Spring MVC Controller용 CORS 설정

Security 필터를 거치지 않는 요청에는 동작

REST API /api/** 경로만 대상으로 설정 가능

둘 다 있어야 하는 이유

Spring Security 적용 시, WebMvcConfigurer만으로는 Security Filter 이전 단계에서 CORS 처리가 안 될 수 있음

그래서 Security랑 MVC 양쪽 모두에서 CORS 설정이 필요함

결론

CorsConfig는 Security 대응용

WebConfig는 일반 MVC 대응용

동시에 있어도 충돌 없음, 각 레이어에서 필요한 설정임

실제로 WebConfig에 PATCH를 추가했어도 여전히 CORS 정책 이슈가 존재했고 관련하여 코드 추가

collabity-backend\src\main\java\com\collabity\web\config\CorsConfig.java 코드 추가

SecurityConfig에서 .cors(Customizer.withDefaults()) < 라는 한 줄 추가

안 된 작업

  • .xlsx 파일 타입명이 길어서 디자인이 깨질 경우 줄바꿈 표시
  • 목록 윗부분에 각 컬럼명이 표시
  • 팀별 저장 공간 용량

@ms-0o0
Copy link
Copy Markdown
Contributor Author

ms-0o0 commented Jan 29, 2026

현재 완료하여서 확인 해주시면 main에 합치겠습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants