Repository navigation
Conversation
chjpooh2002
approved these changes
Nov 9, 2026
Comment on lines
+71
to
+72
| } catch (GoogleLoginException e) { | ||
| throw new BusinessException(AuthErrorCode.GOOGLE_UNAVAILABLE); |
Contributor
There was a problem hiding this comment.
GoogleLoginException을 AUTH502_1로 바꾸면서 로그가 하나도 남지 않아요. 전역 예외 처리도 BusinessException은 로그를 안 남겨서, 구글 장애·클라이언트 시크릿 오류·토큰 엔드포인트 설정 오류가 운영에서 전부 흔적 없이 502로만 보이게 됩니다. 예외 메시지는 GoogleAuthCodeClient에서 이미 민감한 값을 빼고 만들고 있으니, 여기서 log.warn("구글 로그인 실패: {}", e.getMessage()) 정도는 남겨주면 좋을 거 같아요!
Comment on lines
+59
to
+62
| userCommandService.recordLogin(userId); | ||
| String refreshToken = refreshTokenIssuer.issueForNewLogin(userId); | ||
| return AuthLoginResult.loggedIn(jwtTokenProvider.issueAccessToken(userId), refreshToken, userId, | ||
| userQueryService.getNickname(userId)); |
Contributor
There was a problem hiding this comment.
(사소) recordLogin과 getNickname이 각각 회원을 다시 조회해서 로그인 한 번에 같은 회원 조회 쿼리가 두 번 나가요(조건 조회라 1차 캐시를 타지 않음). 급하진 않지만, 연결 지점 모양을 정할 때 recordLogin이 닉네임을 돌려주게 하는 방법도 있을 것 같습니다!
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
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.
관련 이슈
작업 내용
POST /api/v1/auth/login(명세 1번)family_idrecordLogin,getNickname(CLAUDE.md 표 반영)명세와 달라진 점
명세에 없거나 달랐던 부분을
s1-api-spec.md에 함께 반영함AUTH502_1문구를 노션과 같은 "다시 시도해주세요."로 통일refreshToken,Path=/api/v1/auth,SameSite기본Lax(AUTH_COOKIE_SAME_SITE로 변경 가능)provider가 없거나GOOGLE이 아니면AUTH400_1USER404_1테스트
전체
./gradlew build통과AUTH401_3,AUTH502_1, 탈퇴 회원USER404_1code·codeVerifier·redirectUri는COMMON400_1이고 구글을 부르지 않음provider오류AUTH400_1(NAVER, 소문자, null)toString에 토큰 미노출확인 체크리스트
./gradlew build통과리뷰어에게
AUTH200_2는 명세대로 코드 하나에 문구가 2개입니다.ResultCodeRegistry가 중복 코드를 막아서 두 번째 문구만 컨트롤러에서 넣었습니다. 화면은isCompletedSignUp으로 구분해야 합니다.