Skip to content

Develop#286

Merged
gisu1102 merged 5 commits intomainfrom
develop
Oct 16, 2025
Merged

Develop#286
gisu1102 merged 5 commits intomainfrom
develop

Conversation

@gisu1102
Copy link
Copy Markdown
Contributor

@gisu1102 gisu1102 commented Oct 16, 2025

#️⃣ 연관된 이슈

#285

📝 작업 내용

user name update

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 사용자 이름을 업데이트하기 위한 새로운 API 엔드포인트 추가 (PUT /users/name)
    • 사용자 이름 업데이트 요청에 대한 유효성 검사 추가 (1-10자, 공백 제외)
  • 개선사항

    • 쿠키 도메인 처리 로직 개선
    • 사용자 프로필 관리 기능 최적화

@gisu1102 gisu1102 merged commit ac6ed76 into main Oct 16, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 16, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

사용자 이름 업데이트 기능을 추가하고, 닉네임 중복 검사 로직을 단순화하며, JWT 쿠키 도메인 설정 방식을 개선했습니다. 새로운 REST 엔드포인트와 해당 서비스 계층 로직이 도입되었습니다.

Changes

Cohort / File(s) 변경 요약
사용자 이름 업데이트 기능
src/main/java/inu/codin/codin/domain/user/controller/UserController.java, src/main/java/inu/codin/codin/domain/user/dto/request/UserNameUpdateRequestDto.java, src/main/java/inu/codin/codin/domain/user/service/UserService.java, src/main/java/inu/codin/codin/domain/user/entity/UserEntity.java
새로운 PUT /users/name 엔드포인트 추가, 이름 유효성 검증(한글/영문만, 최대 10자), 사용자 이름 업데이트 메서드 구현
닉네임 중복 검사 로직 개선
src/main/java/inu/codin/codin/domain/user/repository/UserRepository.java, src/main/java/inu/codin/codin/common/security/service/AuthCommonService.java, src/main/java/inu/codin/codin/domain/user/service/UserService.java
닉네임 중복 검사 메서드에서 userId 파라미터 제거, 모든 호출 사이트 업데이트
JWT 쿠키 도메인 설정
src/main/java/inu/codin/codin/common/security/service/JwtService.java
BASEURL에서 도메인을 추출하여 x-access-token, x-refresh-token 쿠키 도메인 설정 개선

Sequence Diagram

sequenceDiagram
    participant Client
    participant UserController
    participant UserService
    participant UserRepository
    participant UserEntity
    participant Database

    Client->>UserController: PUT /users/name
    UserController->>UserService: updateUserName(UserNameUpdateRequestDto)
    UserService->>UserService: 현재 사용자 조회
    UserService->>UserService: 이름 유효성 검증<br/>(공백 제거, 한글/영문, 최대 10자)
    alt 이름 변경 필요
        UserService->>UserEntity: updateName(newName)
        UserEntity->>UserEntity: this.name = newName
        UserService->>Database: save(user)
        Database-->>UserService: 저장 완료
        UserService-->>UserController: 성공
    else 이름 변경 없음
        UserService-->>UserController: 스킵
    end
    UserController-->>Client: 200 SingleResponse
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

여러 계층에 걸쳐 새로운 기능이 도입되었으며, 닉네임 중복 검사 로직이 변경되었습니다. 변경사항들이 일관된 패턴을 따르지만, 각 파일마다 서로 다른 목적의 수정이 필요하여 중간 수준의 검토 노력이 요구됩니다.

Possibly related PRs

Poem

🐰 이름을 바꾸는 마법,
닉네임 중복은 피해 가고,
쿠키에 도메인 담아 날리면,
사용자 정보는 더욱 맛깔나네! 🥕✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b7cbbd4 and 3fae328.

📒 Files selected for processing (7)
  • src/main/java/inu/codin/codin/common/security/service/AuthCommonService.java (1 hunks)
  • src/main/java/inu/codin/codin/common/security/service/JwtService.java (2 hunks)
  • src/main/java/inu/codin/codin/domain/user/controller/UserController.java (2 hunks)
  • src/main/java/inu/codin/codin/domain/user/dto/request/UserNameUpdateRequestDto.java (1 hunks)
  • src/main/java/inu/codin/codin/domain/user/entity/UserEntity.java (1 hunks)
  • src/main/java/inu/codin/codin/domain/user/repository/UserRepository.java (1 hunks)
  • src/main/java/inu/codin/codin/domain/user/service/UserService.java (3 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot mentioned this pull request Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant