Skip to content

Conversation

@jsoonworld
Copy link
Member

@jsoonworld jsoonworld commented Aug 10, 2025

📄 Work Description

  • 신규 유저가 소셜 로그인을 시도할 때, 클라이언트 측에서 발생하는 응답 데이터 파싱 오류를 해결했습니다.
  • 문제 원인: SignInResponse DTO에 적용된 @JsonInclude(JsonInclude.Include.NON_NULL) 어노테이션으로 인해, null 값을 가진 필드(accessToken, refreshToken 등)가 JSON 응답에서 제외되었습니다.
  • 해결: 해당 어노테이션을 제거하여, null 값을 가진 필드도 항상 응답에 포함되도록 수정했습니다.

💬 To Reviewers

  • 신규 유저의 소셜 로그인 케이스를 테스트할 때, 응답 JSON 바디에 accessToken, refreshToken, userId 필드가 null 값으로 정상적으로 포함되는지 확인 부탁드립니다.
  • 이번 수정은 신규 유저의 응답 구조만 변경하며, 기존 유저의 로그인 응답에는 영향을 주지 않습니다.

⚙️ ISSUE

@jsoonworld jsoonworld requested a review from Copilot August 10, 2025 00:25
@jsoonworld jsoonworld self-assigned this Aug 10, 2025
@jsoonworld jsoonworld added the ♻️ refactor 코드 리팩토링 ex) 형식변경 label Aug 10, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request fixes a response data parsing issue for new users during social login by removing the @JsonInclude(JsonInclude.Include.NON_NULL) annotation from the SignInResponse DTO. This ensures that fields with null values (like accessToken, refreshToken, and userId) are always included in the JSON response.

  • Removes @JsonInclude(JsonInclude.Include.NON_NULL) annotation and its import
  • Ensures consistent response structure for both new and existing users
  • Fixes client-side parsing errors when handling new user login responses

package org.terning.terningserver.auth.dto.response;

import com.fasterxml.jackson.annotation.JsonInclude;
import org.terning.terningserver.auth.dto.Token;
Copy link

Copilot AI Aug 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a typo in the PR description - '소A셜' should be '소셜'.

Copilot uses AI. Check for mistakes.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot이 오타를 잘 잡아주네요! 😅 꼼꼼한 피드백 감사합니다. 바로 수정했습니다!

@jsoonworld jsoonworld changed the title fix: 신규 유저 로그인 시 응답 데이터 누락 문제 해결 [♻️ refactor/#297] 신규 유저 로그인 시 응답 데이터 누락 문제 해결 Aug 10, 2025
@jsoonworld jsoonworld merged commit 1f08431 into develop Aug 10, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ refactor 코드 리팩토링 ex) 형식변경 size/XS 🦊장순🦊

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[♻️ refactor/#297] 신규 유저 로그인 시 응답 데이터 누락 문제 해결

2 participants