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

Security whilteList 방식 변경 #22

Merged
merged 17 commits into from
Nov 2, 2023
Merged

Security whilteList 방식 변경 #22

merged 17 commits into from
Nov 2, 2023

Conversation

OJOJIN
Copy link
Collaborator

@OJOJIN OJOJIN commented Nov 1, 2023

Related Issue 🪢

Summary 🌿

  • 기존 WhiteList를 SecurityConfig 안에서 ignore를 통해 진행했는데 그렇게 되면 securityfilter 자체를 모두 무시하기에 문제가 생길 수 있고 사용자가 들어오는 api를 확인하는 log를 남길 수 없기에 인증 토큰이 필요한 부분인 JwtAuthenticationFilter 내에서 whiteList 처리를 해주는 것으로 변경

Before i request PR review 🧤

  • 기존에 SecurityConfig에서 whiteList를 추가하는 것이 아닌 JwtAuthenticationFilter 에서 추가하는 것으로 변경되었습니다!

@OJOJIN OJOJIN added fix 서비스 로직에 맞게 코드 변경 진영🐧 진영 라벨 labels Nov 1, 2023
@OJOJIN OJOJIN self-assigned this Nov 1, 2023
@OJOJIN OJOJIN merged commit dff49f3 into develop Nov 2, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 서비스 로직에 맞게 코드 변경 진영🐧 진영 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] Security whitelist 처리 방식 변경
1 participant