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: Security AuthTokenFilter에서 에러 잡고 다음 필터로 넘기도록 설정 변경 #26

Merged
merged 2 commits into from
Mar 3, 2024

Conversation

sooyoungh
Copy link
Member

구현 사항

  • AuthTokenFilter에서 바로 에러 throw하지 않도록 변경 ->인증이 유효하지 않아도 이후 필터로 진행
  • permitAll() 사용하기 위해 변경! 이제 인증 필요없는 것들은 permitAll()로 선언해주면 됩니다!
  • permitAll()은 필터를 안거치는 게(인증을 안하는 게 아니라), 인증이 유효하지 않아도 이후 필터로 계속 진행해서 api 호출한다는 의미입니다
  • 반면 아예 필터단을 안 거치려면 web.ignoring()에 경로 등록해주면 됩니다!

참고 사항

- AuthTokenFilter에서 바로 에러 throw하지 않도록 변경
- permitAll() 사용하기 위해 변경
@sooyoungh sooyoungh self-assigned this Mar 3, 2024
@sooyoungh sooyoungh added 🐞 fix 버그 수정 👻 팻치 @sooyoungh 🔥 H-5 5시간 안에 리뷰해주세요 labels Mar 3, 2024
Comment on lines +64 to +71
.antMatchers("/api/v1/auth/**"
,"/api/v1/admin/**"
, "/api/v1/fcm-push/**"
, "/api/v1/auth/**"
, "/api/v1/admin/**"
, "/api/v1/fcm-push/random-spot/**"
, "/api/v1/heendy-guide/**"
, "/websocket/**").permitAll()
Copy link
Member Author

Choose a reason for hiding this comment

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

인증 필요없는 경로들 추가

@sooyoungh sooyoungh merged commit 3f592ae into dev Mar 3, 2024
2 checks passed
@sooyoungh sooyoungh deleted the HEENDY-89-Security-filter branch March 3, 2024 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 fix 버그 수정 👻 팻치 @sooyoungh 🔥 H-5 5시간 안에 리뷰해주세요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant