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] bug refresh token 오류 #396

Closed
wants to merge 1 commit into from
Closed

Conversation

AYoungSn
Copy link
Contributor

@AYoungSn AYoungSn commented Dec 15, 2023

📌 개요

  • refresh token 생성 로직 수정

💻 작업사항

  • refresh token 으로 access token 재 요청시 refresh token 에 담긴 값이 유저마다 고유하지 않을 수 있는 문제로 인해 유저 로그인이 다른 사용자로 바뀌는 문제가 있어서 refresh token 생성 로직 수정함

💡Issue 번호

@AYoungSn AYoungSn self-assigned this Dec 15, 2023
@AYoungSn AYoungSn changed the base branch from main to dev December 15, 2023 06:28
Copy link

codecov bot commented Dec 15, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (617ffbf) 68.77% compared to head (1854b92) 68.64%.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev     #396      +/-   ##
============================================
- Coverage     68.77%   68.64%   -0.14%     
+ Complexity      794      793       -1     
============================================
  Files           252      252              
  Lines          3629     3629              
  Branches        347      347              
============================================
- Hits           2496     2491       -5     
- Misses          942      944       +2     
- Partials        191      194       +3     
Flag Coverage Δ
integrationTest 68.58% <ø> (-0.14%) ⬇️
unitTest 8.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AYoungSn AYoungSn closed this Dec 15, 2023
@AYoungSn AYoungSn deleted the 394-bug-refresh-token-오류 branch December 15, 2023 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 [Bug] Refresh Token 오류
1 participant