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: 스프링 시큐리티 기본 설정 및 토큰 기능 구현 #13

Merged
merged 8 commits into from
Jun 4, 2024

Conversation

devmizz
Copy link
Contributor

@devmizz devmizz commented May 31, 2024

😋 작업한 내용

  • 스프링 시큐리티 세팅
  • JWT Filter, Generator 구현
  • GlobalExceptionFilter 수정

🙏 PR Point

  • JWTProcessor의 네이밍이 마음에 안 들어서... 추천해주실 수 있으면 그걸로 수정해보겠습니다!
  • 추가한 코드가 너무 많네요... 늘 PR 사이즈가 커서 고민입니다. 잘 잘라볼게요!

👍 관련 이슈


@devmizz devmizz added the feat add feature label May 31, 2024
@devmizz devmizz self-assigned this May 31, 2024
Copy link
Contributor

@GaBaljaintheroom GaBaljaintheroom left a comment

Choose a reason for hiding this comment

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

전반적으로 저한텐 너무 깔끔하고 이해하기 쉬운 코드였어요.
JWT를 잘 알진 못하지만 쉽게 이해할 수 있었습니다.

제가 이해한 JWT 흐름이 맞는지 확인해주시면 감사하겠습니다.

  1. 로그인 -> accessToken, refreshToken 발급
  2. accessToken 만료시 refreshToken 으로 accessToken 재발급 (이때, refreshToken도 재발급)

@devmizz devmizz force-pushed the feat-setting-security branch from c346b6d to 2ea5105 Compare June 2, 2024 23:25
@devmizz devmizz force-pushed the feat-setting-security branch from 2ea5105 to 421813f Compare June 3, 2024 16:03
Copy link
Contributor

@GaBaljaintheroom GaBaljaintheroom left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다!!

@devmizz devmizz merged commit 1103652 into develop Jun 4, 2024
1 check passed
@devmizz devmizz deleted the feat-setting-security branch June 4, 2024 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat add feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants