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

Firebase Auth Filter가 선택된 URL에 대해서만 적용되도록 합니다. #175

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

litsynp
Copy link
Member

@litsynp litsynp commented Jun 26, 2023

📌 개요 (필수)

Firebase Auth Filter가 선택된 URL에 대해서만 적용되도록 합니다.

Resolves #163.


🔨 작업 사항 (필수)

Firebase Auth Filter가 선택된 URL에 대해서만 적용되도록 합니다.

해결 방법:

  • @Component로 컴포넌트를 등록하지 않고 생성자를 호출해 인증 필터로 주입

  • 인증 필터에 대해 WebSecurityCustomizer를 만들어 whitelist를 적용

추가로, Swagger에서 버튼을 눌러 Bearer Authorization header를 넣을 수 있도록 했습니다.

image


🌱 연관 내용 (선택)

원래 원했던 것은 커스텀 어노테이션을 API 위에 붙여 인증하는 것으로 알고 있는데.. 아직 구현이 되어 있지 않습니다.

현재 방식을 그대로 유지한다면 이 글을,

헤더에서 빼오는 방식으로 커스텀 어노테이션을 사용하고 싶다면 이 글을 참고해도 좋을 것 같습니다. 여기에 기능이 좀 더 필요하다면(role 검사) 좀 더 작업이 필요하겠지만요!

@litsynp litsynp added 🐛 Bug 버그 ✨ Feature 기능 개발 labels Jun 26, 2023
@litsynp litsynp requested a review from a team June 26, 2023 12:07
@litsynp litsynp self-assigned this Jun 26, 2023
@github-actions
Copy link

Test Results

21 tests   21 ✔️  4s ⏱️
  8 suites    0 💤
  8 files      0

Results for commit 52a5a88.

@litsynp
Copy link
Member Author

litsynp commented Jun 26, 2023

프론트엔드에서 웹뷰에서 토큰을 넘길 준비가 되면 머지하도록 합니다.

Copy link
Member

@JoeCP17 JoeCP17 left a comment

Choose a reason for hiding this comment

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

옹 이런식으로 바꾸셨군요! 좋습니당 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug 버그 ✨ Feature 기능 개발
Projects
Status: In Pull Request
Development

Successfully merging this pull request may close these issues.

[setting] swagger에 JWT security 추가
2 participants