feat: 회원 프로필 수정 API 및 관련 로직 추가 - #28
Conversation
- MemberProfileUseCase에 프로필 수정 기능 및 구현 로직 추가 - 프로필 수정 API(`/me/profile`) Controller, Service, Repository 계층에 구현 - 프로필 수정 시 닉네임 중복 및 빈 프로필 이미지 URL 검증 로직 추가 - 테스트(MemberProfileServiceTest, MemberControllerTest)로 수정 동작 검증 - API 문서화 및 RestDocs 테스트(MemberProfileDocsTest) 추가
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5a7f83e316
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| @field:NotBlank | ||
| @field:Size(min = 2, max = 40) | ||
| @field:Pattern(regexp = "^[가-힣a-zA-Z0-9_-]+$") |
There was a problem hiding this comment.
MemberProfileUpdateRequest.nickname에 적용된 정규식(^[가-힣a-zA-Z0-9_-]+$)은 공백을 포함한 입력을 즉시 400으로 거절하므로, 서비스의 request.nickname.trim() 정규화 로직이 실제 API 경로에서 실행되지 않습니다. 이 커밋의 문서/로직은 앞뒤 공백 제거 저장을 의도하고 있어(예: " unit_user "), 현재 구현은 의도와 동작이 불일치합니다. 닉네임 검증을 trim 이후 값 기준으로 수행하거나, 앞뒤 공백을 허용한 뒤 서비스에서 정규화하도록 검증 규칙을 조정하는 것이 안전합니다.
Useful? React with 👍 / 👎.
1 participant
📌 작업 개요
회원 프로필 수정 API 및 관련 로직 추가
🔗 관련 이슈
✨ 변경 사항
/me/profile) Controller, Service, Repository 계층에 구현🧪 테스트
✅ 체크리스트
feature/,fix/,hotfix/등)💬 리뷰어에게
@codex