[REFACTOR] OAUTH2 로그인 리다이렉트 URL 동적으로 추가하는 로직 개발 - Githubissues

Kernel360 / F2-BIENGUAL-BE

영어 학습 플랫폼 - 백엔드 리포지토리(추후 수정 예정)

2 stars 1 forks source link

[REFACTOR] OAUTH2 로그인 리다이렉트 URL 동적으로 추가하는 로직 개발 #216

Closed mooncw closed 4 days ago

mooncw commented 5 days ago

PR 타입 (하나 이상의 PR 타입을 선택해주세요)

[x] 기능 추가
[ ] 기능 삭제
[ ] 버그 수정
[ ] 문서 수정
[x] 리팩토링
[ ] 스타일 수정
[ ] 테스트 코드 추가
[ ] 의존성, 환경 변수, 빌드 관련 코드 업데이트
[ ] 개발 환경 세팅

변경 사항

기존에 있었지만 사용안하던 OAuth2AuthorizationRequestBasedOnCookieRepository 사용
추가로 프론트에서 보낸 returnUrl을 받아 쿠키에 저장하여 OAuth2SuccessHandler에서 사용하는 방식으로 구현

참고 사항

관련 이슈 번호: #215
원래 이인제 디렉터님 말씀처럼 URL로만 returnUrl을 들고 있다가 OAuth2SuccessHandler에서 가져오는 방식을 하려고 했지만, 그렇게 구현하기엔 시간이 많이 소요될 수도 있고, 가능하다고 100% 확신할 수가 없어서 쿠키 방식을 사용했습니다. (디렉터님 말씀을 제가 잘못 이해한 거일 수도 있지만요.)
OAuth2AuthorizationRequestBasedOnCookieRepository 와 OAuth2SuccessHandler 에 쿠키를 받아오는 메서드가 있는데, 이것을 CookieUtil에 둘지 말지 의견을 주시면 감사하겠습니다.

셀프 체크리스트

[x] 코드가 정상적으로 동작하는지 확인
[ ] 새로운 테스트를 추가했거나 기존 테스트를 통과하는지 확인
[x] 코드 스타일 가이드에 맞게 포맷팅했는지 확인
[ ] 관련 문서를 업데이트했는지 확인

mooncw commented 5 days ago

쿠키에 기존 소셜 로그인 리다이렉트와 프론트에서 요청한 리다이렉트를 각각 저장하고 이후에 성공하면 이를 합쳐서 redirect 하는 것으로 이해했는데 혹시 잘못된 부분이 있을까요? 쿠키 받아오는 부분은 앞으로 더 추가되지 않는다면 지금처럼 써도 괜찮지 않을까 싶습니다

고생하셨습니다

쿠키에 저장되는 리다이렉트는 상대URL 이고, OAuth2SuccessHandler에서 yml파일에서 가져온 기저URL과 쿠키에서 가져온 상대URL을 합칩니다. 즉 말씀하신 기존 소셜 로그인 리다이렉트는 쿠키에 저장하지 않습니다.

tigris24 commented 5 days ago

쿠키에 기존 소셜 로그인 리다이렉트와 프론트에서 요청한 리다이렉트를 각각 저장하고 이후에 성공하면 이를 합쳐서 redirect 하는 것으로 이해했는데 혹시 잘못된 부분이 있을까요? 쿠키 받아오는 부분은 앞으로 더 추가되지 않는다면 지금처럼 써도 괜찮지 않을까 싶습니다 고생하셨습니다

쿠키에 저장되는 리다이렉트는 상대URL 이고, OAuth2SuccessHandler에서 yml파일에서 가져온 기저URL과 쿠키에서 가져온 상대URL을 합칩니다. 즉 말씀하신 기존 소셜 로그인 리다이렉트는 쿠키에 저장하지 않습니다.

답변 감사합니다 덕분에 이해가 되었습니다