Skip to content

Conversation

@KoreaNirsa
Copy link
Contributor

@KoreaNirsa KoreaNirsa commented Sep 3, 2025

수정 사유 Reason for modification

  • 버그수정 Bug fixes
  • 기능개선 Enhancements
  • 기능추가 Adding features
  • 기타 Others

수정된 소스 내용 Modified source

JWT 유틸 분리, 파일 업로드 서비스 이전, Swagger 호환 List 적용

  • extractUserFromJwt(...) 메서드를 JwtTokenUtil로 이동하고 컨트롤러 사용부 수정
  • XssUtil 생성 및 컨트롤러에 빈 임시 주입(후속 리팩토링 시 제거 예정)
  • 첨부파일 업로드/파싱 로직을 컨트롤러 → 서비스 계층으로 이전
  • Swagger 지원을 위해 컨트롤러 시그니처 변경
  • 요청 DTO와 비즈니스/도메인 객체 분리로 책임 명확화

JUnit 테스트 JUnit tests

  • JUnit 테스트 JUnit tests
  • 수동 테스트 Manual testing

테스트 브라우저 Test Browser

  • Chrome
  • Firefox
  • Edge
  • Safari
  • Opera
  • Internet Explorer
  • 기타 Others

테스트 스크린샷 또는 캡처 영상 Test screenshots or captured video

적용 전

image image

적용 후

image image image image

…agger 호환 List<MultipartFile> 적용

 - extractUserFromJwt(...) 메서드를 JwtTokenUtil로 이동하고 컨트롤러 사용부 수정
 - XssUtil 생성 및 컨트롤러에 빈 임시 주입(후속 리팩토링 시 제거 예정)
 - 첨부파일 업로드/파싱 로직을 컨트롤러 → 서비스 계층으로 이전
 - Swagger 지원을 위해 컨트롤러 시그니처 변경
   - MultipartHttpServletRequest → @RequestParam List<MultipartFile> files
   - 프론트에서 다중 파일 키를 file_ → files로 요청하도록 로직 변경 [PR 병합 대기]
   - 변경 영향으로 현재 insert는 파일 1개만 업로드되는 상태 (해당 PR 머지 후 다중 업로드 정상화 예정)
 - 요청 DTO와 비즈니스/도메인 객체 분리로 책임 명확화
@eGovFrameSupport
Copy link
Contributor

표준프레임워크에 대한 지속적인 참여에
대단히 감사드립니다.

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