Skip to content

Conversation

@m-a-king
Copy link
Collaborator

@m-a-king m-a-king commented Dec 7, 2024

resolved :

📌 과제 설명

쿠키에서 accessToken을 추출할 수 있습니다.

👩‍💻 요구 사항과 구현 내용

  • 쿠키와 헤더 모두에서 토큰을 추출할 수 있음.

    • 쿠키에서 토큰을 초기화하지 못하면 헤더에서 토큰을 찾음.
    • 헤더에서 토큰을 찾는 이유는 스웨거, 디버깅.
  • accessToken 리팩토링.

  • 쿠키 NPE 예방.

✅ PR 포인트 & 궁금한 점

accessToken 초기화 흐름에 집중해 주세요!

@m-a-king m-a-king linked an issue Dec 7, 2024 that may be closed by this pull request
2 tasks
@m-a-king m-a-king self-assigned this Dec 7, 2024
Copy link
Collaborator

@ayoung-dev ayoung-dev left a comment

Choose a reason for hiding this comment

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

👍🏻

Copy link
Collaborator

@leebs0521 leebs0521 left a comment

Choose a reason for hiding this comment

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

고생하셨씁니당!

- 쿠키와 헤더 모두에서 토큰을 추출할 수 있음.
  - 쿠키에서 토큰을 초기화하지 못하면 헤더에서 토큰을 찾음.
  - 헤더에서 토큰을 찾는 이유는 스웨거, 디버깅.

- accessToken 리팩토링

- 쿠키 NPE 예방.
@m-a-king m-a-king force-pushed the feature/182-add-cookie-auth branch from c5f80cf to efc7843 Compare December 7, 2024 12:03
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 7, 2024

@m-a-king m-a-king merged commit d9661c5 into main Dec 7, 2024
3 checks passed
@m-a-king m-a-king deleted the feature/182-add-cookie-auth branch December 7, 2024 12:07
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.

[FEATURE] 쿠키 기반 인증

4 participants