카카오 redirect url 수정 #13
Open
seung721 wants to merge 46 commits into
Open
Conversation
# Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
# 변경점 👍 새로 구현한 기능 및 주요 변경점 여유로운길, 발견하는길 구현완료, 바른길 수정필요해보임 (따로 수정해서 다시 올리겠습니다) # 리팩토링 💧 출발, 목적지를 선택하면 바른길, 여유로운길, 발견하는길 셋 중 선택할 수 있고 경로 도중 여유로운길, 발견하는길의 경유지에는 따로 마커를 띄워서 경유지 확인이 쉽게 구현했습니다. # 테스트 💻 변경점을 테스트하기 위한 방법 기술 출발, 목적지 설정 후 원하는 경로추천을 선택하여 테스트해봅니다. # 스크린샷 🖼 변경된 부분에 대한 스크린샷 [onmywayPreview.pdf](https://github.com/user-attachments/files/27943106/onmywayPreview.pdf) 사용하지 않은 항목은 모두 지워주세요. --------- Co-authored-by: Seogyeong Yun <gsm09157@naver.com> Co-authored-by: scholar-star <cheve1224@pusan.ac.kr> Co-authored-by: ziun <jiwun3041@gmail.com>
# 변경점 👍 * 로그인이 성공하면 요청한 URL로 이동하게끔 하는 리다이렉션 로직을 구현했습니다. # 버그 해결 💊 * 처음에 지정한 리다이렉션 URL로 넘기는 테스트를 진행할 때, 무한으로 리다이렉션되는 문제가 있었습니다. * kakao api의 redirect url을 제대로 허용했는지 확인하고, 홈 화면 url에 대해 authenticated를 permitAll로 푸니 해결되었습니다. # 테스트 💻 * 블랙박스 테스트로 카카오 로그인을 직접 시행하여 작동 여부를 확인하였습니다. * 그와 동시에 로깅 모드를 debug로 바꿔 자세한 흐름을 살펴보았습니다. # 비고 ✏ * 지정하지 않았을 때의 url은 홈 화면으로 설정하여야 합니다. --------- Co-authored-by: Seogyeong Yun <gsm09157@naver.com>
# 변경점 👍 * 네이버 API를 연결한 후, 이를 이용해 이미지를 검색하여 URL을 가지고 오는 로직을 만들었고, 이를 DTO에 통합시켰습니다! * URL을 가지고 오는 로직에는 기본적으로 시간이 소요되는 편이기 때문에, 서경님이 만드신 RecommendService 로직의 반환값의 형태를 비동기에 맞게 바꾸었어요. # 버그 해결 💊 * 비동기 형식을 사용하지 않을 때는 이미지까지 가져오는 데 7~8초가 걸리던 문제가 발생했습니다. * 그래서, 비동기 처리 의존성인 Spring Webflux를 도입, 거기에 있는 Mono 반환값을 사용하여 비동기 형식으로 데이터를 처리하게 했더니 DTO 호출 시간이 기존보다 절반 이상 감소하였습니다. # 테스트 💻 * Postman으로 Black box test(호출-결과 위주로 보는 test)를 진행하였습니다. # 스크린샷 🖼 <img width="2880" height="1800" alt="화면 캡처 2026-05-17 163022" src="https://github.com/user-attachments/assets/dc1716c0-c056-4156-adfe-94cf5b57df20" /> * 해당 형식으로 DTO가 반환됩니다. --------- Co-authored-by: Seogyeong Yun <gsm09157@naver.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
변경점 👍
+) frontend/ganrungil/pages/find-route안에 있는 한글이름.jsx 두개의 파일은 제가 빠른 시일내에 없애도록 하겠습니다. 현재 길 찾는 로직을 들고오다보니 저 파일두개를 가져올 수밖에 없었습니다. test-deploy 브랜치는 저 두파일은 사용하지 않고 있습니다(두 jsx파일은 그저 로컬 개발을 위한 참고용입니다.)
버그 해결 💊
테스트 💻
헤더에 로그인 누르면 로그인 페이지(/signup)로 이동합니다.
카카오 로그인 버튼 누르면 카카오 로그인 페이지로 이동합니다.