Skip to content

[Feat] 유저 프로필 조회 시 이메일 추가 #138

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pooreumjung merged 1 commit into from
May 27, 2026
Merged

[Feat] 유저 프로필 조회 시 이메일 추가 #138

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions src/main/java/com/semosan/api/domain/user/dto/response/GetUserProfileResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@ public record GetUserProfileResponse(
Double height,
Double weight,
ExerciseType exerciseType,
LocalDate birthDate
LocalDate birthDate,
String email

@gemini-code-assist gemini-code-assist Bot May 27, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

high

사용자 프로필 조회 응답에 이메일이 추가되었습니다. 이메일은 개인정보에 해당하므로 API 응답에 포함될 때 주의가 필요합니다.

이 GetUserProfileResponse DTO가 오직 인증된 사용자 본인의 정보를 조회하는 경우에만 사용된다면 문제가 없지만, 만약 다른 사용자의 프로필을 조회하는 데에도 사용된다면 개인정보가 노출될 수 있는 보안 취약점이 될 수 있습니다.

이 응답이 다른 사용자에게 노출될 가능성이 있다면, 이메일 필드를 제거하거나 다른 사용자를 위한 별도의 공개용 프로필 DTO를 만드는 것을 고려해 주세요. 예를 들어, GetPublicUserProfileResponse 와 같이 민감한 정보를 제외한 DTO를 분리하는 것이 좋은 방법이 될 수 있습니다.

) {
// User와 온보딩 정보로 프로필 조회 응답 DTO를 생성합니다.
public static GetUserProfileResponse of(User user, UserOnboarding userOnboarding) {
Expand All @@ -39,7 +40,8 @@ public static GetUserProfileResponse of(User user, UserOnboarding userOnboarding
user.getHeight(),
user.getWeight(),
exerciseType,
user.getBirthDate()
user.getBirthDate(),
user.getEmail()
);
}
}