-
Notifications
You must be signed in to change notification settings - Fork 7
[BM-246] 로그인 요청 url의 리다이렉트 url로 로그인시 리다이렉트 #111
Conversation
Codecov Report
@@ Coverage Diff @@
## main #111 +/- ##
============================================
- Coverage 81.62% 79.53% -2.09%
Complexity 364 364
============================================
Files 74 75 +1
Lines 1132 1163 +31
Branches 27 32 +5
============================================
+ Hits 924 925 +1
- Misses 190 220 +30
Partials 18 18
Continue to review full report at Codecov.
|
📊 BidMarket-pr-111 분석 결과 확인 링크 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏼
for (Cookie cookie : cookies) { | ||
if (cookie.getName().equals(name)) { | ||
return Optional.of(cookie); | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for (Cookie cookie : cookies) { | |
if (cookie.getName().equals(name)) { | |
return Optional.of(cookie); | |
} | |
} | |
return Arrays.stream(cookies) | |
.filter(cookie -> cookie.getName().equals(name)) | |
.findFirst(); |
스트림쓰면 더 가독성있게 쓸 수 있을것 같습니다
📊 BidMarket-pr-111 분석 결과 확인 링크 |
📊 BidMarket-pr-111 분석 결과 확인 링크 |
📊 BidMarket-pr-111 분석 결과 확인 링크 |
기존에 하드코딩으로 되어있던 리다이렉트 url을 프론트에서 처음 요청시 redirect url로 리다이렉트 하도록 수정했습니다.
localhost:8080/oauth2/authorization/google?redirect_uri=http://localhost:3000/auth
이렇게 로그인 요청시 localhost:3000/auth로 리다이렉트 해줌