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] JWT Token 인증 방식 변경 #23

Merged
merged 3 commits into from
Jan 23, 2024
Merged

[FIX] JWT Token 인증 방식 변경 #23

merged 3 commits into from
Jan 23, 2024

Conversation

PgmJun
Copy link
Contributor

@PgmJun PgmJun commented Jan 23, 2024

✒️ 관련 이슈번호

🔑 Key Changes

  1. Cookie를 통한 JWT 토큰 인증 방식에서 Authorization Header 사용하도록 변경
  2. 같은 같은 계정으로 OAuth2 회원가입 시도 시 발생하는 ConflictException을 ExceptionControllerAdvice에서 처리하도록 구현

📢 To Reviewers

httponly와 secure 설정을 한 Cookie를 활용하는 방법도 안전한 방식이지만,
토큰 인증방식에서 표준화 되어있는 Authorization 헤더를 사용하도록 변경하였습니다.

토큰을 인증하는 방식에 있어서 표준화 되어잇는 Authorization 헤더를 사용하는 것이
일반적으로 개발자가 예측 가능한 방식이기 때문에 빠른 코드 이해를 돕고 이를 통해 개발의 생산성가독성을 향상시킬 수 있는 이점을 얻는 방향이라고 생각되어 이렇게 변경하였습니다.

@PgmJun PgmJun added the Fix label Jan 23, 2024
@PgmJun PgmJun self-assigned this Jan 23, 2024
@PgmJun PgmJun added the Feat label Jan 23, 2024
@PgmJun PgmJun merged commit 68424d5 into develop Jan 23, 2024
@PgmJun PgmJun deleted the feat/#8 branch January 23, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] JWT 기반 소셜 로그인 구현
1 participant