Skip to content

Conversation

@haeun9634
Copy link
Contributor

No description provided.

@haeun9634 haeun9634 linked an issue Dec 4, 2025 that may be closed by this pull request
@haeun9634 haeun9634 merged commit 145be65 into main Dec 4, 2025
1 check passed
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

🤖 GPT PR Review

결론: request-changes

요약:

주요 지적

  • [blocker] CORSConfiguration.java:15 — undefined
    • 근거: CORS 설정에서 허용된 도메인이 하드코딩되어 있습니다.
    • 제안: 허용된 도메인을 환경 변수나 설정 파일에서 관리하도록 변경하여 보안을 강화하세요.
  • [major] UserController.java:42 — undefined
    • 근거: 유저 정보 조회 API에서 예외 처리가 부족합니다.
    • 제안: 유저 정보 조회 시 발생할 수 있는 예외를 처리하고 적절한 HTTP 상태 코드를 반환하도록 수정하세요.
  • [minor] UserService.java:30 — undefined
    • 근거: 유저 정보 조회 API에 대한 테스트가 없습니다.
    • 제안: 유저 정보 조회 API에 대한 단위 테스트를 추가하여 기능이 올바르게 작동하는지 확인하세요.

제안 테스트

  • UserControllerTest.java - 유저 정보 조회 API에 대한 테스트 케이스 추가

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.

[FEAT] 사용자 조회 API 및 WebSecurity 설정 수정

2 participants