Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEAT] Spring security 관련 커스텀 어노테이션 생성 및 적용 #257

Merged
merged 3 commits into from
Aug 27, 2024

Conversation

SSung023
Copy link
Contributor

PR 타입(하나 이상의 PR 타입을 선택해주세요)

☑ 기능 추가
□ 기능 삭제
□ 버그 수정
□ 의존성, 환경 변수, 빌드 관련 코드 업데이트


반영 브랜치

feat/248-auth-custom-annotationmain


변경 사항

@AuthenticationPrincipal 어노테이션의 중복 코드를 제거하기 위해 @GitGetUser 커스텀 어노테이션을 작성하고 적용합니다.

작업 상세 내용

  • @GitGetUser 커스텀 어노테이션 작성
  • 컨트롤러 코드 중, @AuthenticationPrincipal 어노테이션 대신 @GitGetUser 어노테이션으로 변경

테스트 결과

image


연관된 이슈

#248


리뷰 요구사항(선택)

- AuthenticationPrincipal 의 값을 받을 수 있는 커스텀 어노테이션 개발
- @AuthenticationPrincipal 어노테이션이 적용되어 있는 곳을 @gitgetuser 어노테이션으로 변경
- @AuthenticationPrincipal 어노테이션 있던 곳에 커스텀 어노테이션(@gitgetuser) 적용
@SSung023 SSung023 added 💡feature 기능 구현, 개선 사항 관련 ⚙️ refactor 리팩토링 🧑🏻‍💻 BE 백엔드 관련 코드 labels Aug 26, 2024
@SSung023 SSung023 self-assigned this Aug 26, 2024
@SSung023 SSung023 linked an issue Aug 26, 2024 that may be closed by this pull request
2 tasks
Copy link

Test Results

184 files  184 suites   45s ⏱️
296 tests 296 ✅ 0 💤 0 ❌
304 runs  304 ✅ 0 💤 0 ❌

Results for commit 97244a2.

Copy link
Contributor

@kimdozzi kimdozzi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ㅎㅎ

@SSung023 SSung023 merged commit a6912a0 into main Aug 27, 2024
2 checks passed
@SSung023 SSung023 deleted the feat/248-auth-custom-annotation branch August 27, 2024 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧑🏻‍💻 BE 백엔드 관련 코드 💡feature 기능 구현, 개선 사항 관련 ⚙️ refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] Spring security 관련 커스텀 어노테이션 생성 및 적용
2 participants