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

[FIX] jwtAuthenticationFilter의 검증 로직 수정 #40

Merged
merged 1 commit into from
Jan 17, 2025
Merged

Conversation

hyunw9
Copy link
Contributor

@hyunw9 hyunw9 commented Jan 17, 2025

Related Issue 🚀

Work Description ✏️

Jwks 요청 시 발생하는 에러를 수정했습니다.
기존에 Header.Server를 체크하고 있었는데, 필요 없는 로직이 에러를 발생시키고 있어서 삭제했습니다.

@hyunw9 hyunw9 added the 🔨 fix 버그를 발견하여 코드를 수정한 경우(spring boot 내의 기능 코드) label Jan 17, 2025
@hyunw9 hyunw9 self-assigned this Jan 17, 2025
Copy link

height bot commented Jan 17, 2025

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@sung-silver sung-silver self-requested a review January 17, 2025 17:55
Copy link
Contributor

@sung-silver sung-silver left a comment

Choose a reason for hiding this comment

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

아 이거 충돌나겠네요 머지해주시면 제꺼에서 pull 받고 날릴게요~

@sung-silver sung-silver merged commit 9cd43e2 into dev Jan 17, 2025
1 check passed
@sung-silver sung-silver deleted the fix/#39 branch January 17, 2025 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 fix 버그를 발견하여 코드를 수정한 경우(spring boot 내의 기능 코드) size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] Jwks 요청 시 400 에러를 수정합니다.
2 participants