Skip to content

[YS-48] feat: security 관련 설정 추가#6

Merged
chock-cho merged 4 commits intodevfrom
feature/YS-48
Dec 29, 2024
Merged

[YS-48] feat: security 관련 설정 추가#6
chock-cho merged 4 commits intodevfrom
feature/YS-48

Conversation

@Ji-soo708
Copy link
Member

@Ji-soo708 Ji-soo708 commented Dec 28, 2024

💡 작업 내용

  • CORS 설정 추가
  • security 관련 설정 파일 추가
  • ExceptionHandler 추가

✅ 셀프 체크리스트

  • PR 제목을 형식에 맞게 작성했나요?
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있나요?
  • 테스트는 잘 통과했나요?
  • 빌드에 성공했나요?
  • 본인을 assign 해주세요.
  • 해당 PR에 맞는 label을 붙여주세요.

🙋🏻‍ 확인해주세요

  • feat: 인증/인가를 위한 JWT 토큰 발급 로직 구현 #4 에서 같이 올라가야 했을 security 파일들을 추가했습니다
  • 수정님이 이전에 구현해 놓은 예외 로직들 보면서 제가 이해한 대로 ExceptionHandler 파일을 추가했는데 맞게 구현이 되었는지 더블체크 부탁드립니다~

@Ji-soo708 Ji-soo708 added the ✨ FEATURE 기능 추가 label Dec 28, 2024
@Ji-soo708 Ji-soo708 self-assigned this Dec 28, 2024
@Ji-soo708 Ji-soo708 requested a review from chock-cho December 28, 2024 12:23
@sonarqubecloud
Copy link

Copy link
Member

@chock-cho chock-cho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코드가 깔끔하게 작성되었고, 설계와 구현이 명확하게 잘 되어있어 감탄했습니다!
추가적으로 건드릴 부분 없이 LGTM 입니다 ☺️

@chock-cho chock-cho merged commit 32a94f4 into dev Dec 29, 2024
1 check passed
@chock-cho chock-cho deleted the feature/YS-48 branch December 29, 2024 17:12
Ji-soo708 added a commit that referenced this pull request Jan 26, 2025
* style: rename fileName

* feat: add AuthorizationException

* feat: add security configuration

* feat: add ExceptionHandler
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ FEATURE 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants