Skip to content

fix: deny unmatched security routes - #176

Merged
Hyunstone merged 4 commits into
developfrom
feat/security-fail-closed
Jul 22, 2026
Merged

fix: deny unmatched security routes#176
Hyunstone merged 4 commits into
developfrom
feat/security-fail-closed

Conversation

@Hyunstone

@Hyunstone Hyunstone commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

해결하려던 문제 혹은 구현하려는 기능

  • 명시적으로 허용하거나 권한을 부여하지 않은 경로의 기본 정책을 permitAll에서 denyAll로 변경했습니다.
  • Cloud Run 헬스 체크를 위해 GET /actuator/health만 공개하고 다른 Actuator 경로는 차단했습니다.
  • 사용하지 않는 org.bouncycastle:bcpkix-jdk15on:1.70 직접 의존성을 제거했습니다.

테스트 시나리오

  • sh gradlew test --rerun-tasks --no-daemon
    • BUILD SUCCESSFUL
    • SecurityConfigTest 13건을 포함한 전체 테스트 성공
  • sh gradlew dependencyInsight --dependency bcpkix-jdk15on --configuration runtimeClasspath --no-daemon
    • runtimeClasspath에서 일치하는 의존성 없음
  • git diff --check origin/develop...HEAD
    • 오류 없음
  • GitHub Actions CI 성공

Risk

  • 명시되지 않은 경로는 비인증 요청에 401, 인증된 요청에 403을 반환합니다. 의도한 fail-closed 동작입니다.
  • Actuator 인가 테스트는 동일 경로의 테스트용 컨트롤러로 matcher 동작을 검증합니다. 실제 Actuator 빈을 통한 통합 검증은 포함하지 않습니다.

관련 이슈

  • 연결된 이슈 없음

@Hyunstone Hyunstone self-assigned this Jul 22, 2026
@Hyunstone
Hyunstone force-pushed the feat/security-fail-closed branch from 363752e to af41cad Compare July 22, 2026 05:57
@Hyunstone
Hyunstone marked this pull request as ready for review July 22, 2026 06:06
@Hyunstone
Hyunstone merged commit 5dd94ab into develop Jul 22, 2026
2 checks passed
@Hyunstone
Hyunstone deleted the feat/security-fail-closed branch July 22, 2026 06:06
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