Skip to content

Conversation

@GoGradually
Copy link
Collaborator

변경된 점

  • api 인증 필터링에 걸리지 않는 API에 대하여, 모든 버전들을 허용하는 exclude 필터 추가
  • api 버전 변경 시 주의할 점 추가

Copilot AI review requested due to automatic review settings December 31, 2025 08:36
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

이 PR은 API 게이트웨이에 API 버전 넘버링 지원을 추가합니다. 주요 목적은 /v1/login, /v2/signup 등과 같은 버전이 포함된 경로에서도 인증 필터가 올바르게 동작하도록 하는 것입니다.

주요 변경사항:

  • Spring Security 설정에 버전 와일드카드 패턴(/*/login, /*/signup, /*/refresh, /*/login/**)을 추가하여 모든 API 버전에서 인증 제외 경로가 작동하도록 함
  • README.md에 새 API 버전 추가 시 확인해야 할 체크리스트 추가

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/main/java/me/pinitgateway/security/SecurityConfig.java auth host에서 인증이 필요 없는 경로에 대해 버전별 패턴 추가 (/*/login, /*/signup, /*/refresh, /*/login/**)
README.md API 버전 추가 시 확인해야 할 주의사항 문서화 (인증 필터링, 프론트엔드 호환성 체크리스트)

Comment on lines +120 to +123
"/*/login",
"/*/signup",
"/*/refresh",
"/*/login/**",
Copy link

Copilot AI Dec 31, 2025

Choose a reason for hiding this comment

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

문제점:
/*/signup/**/*/refresh/** 패턴이 누락되어 있습니다. 현재 /*/login/**은 추가되었지만 (라인 123), signup과 refresh에는 하위 경로 패턴이 없습니다.

영향:

  • /v1/signup/confirm이나 /v2/refresh/token 같은 하위 경로가 있는 경우, 버전이 붙은 경로에 대해 인증 필터가 적용될 수 있습니다.
  • 기존 /login/** 패턴의 버전별 대응인 /*/login/**를 추가한 것처럼, /signup/**/refresh/**에도 버전별 대응이 필요합니다.

수정 제안:
라인 121과 122 다음에 /*/signup/**/*/refresh/** 패턴을 추가하여 일관성을 유지하세요.

Copilot generated this review using guidance from repository custom instructions.
@GoGradually GoGradually merged commit 0e8e423 into master Dec 31, 2025
7 checks passed
@GoGradually GoGradually deleted the feat/api-버전-넘버링-적용 branch December 31, 2025 08:49
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.

2 participants