Skip to content

feat: NoHandlerFoundException 설정 추가#336

Merged
Soundbar91 merged 3 commits intodevelopfrom
feat/not-found-resource
Mar 1, 2026
Merged

feat: NoHandlerFoundException 설정 추가#336
Soundbar91 merged 3 commits intodevelopfrom
feat/not-found-resource

Conversation

@Soundbar91
Copy link
Collaborator

🔍 개요


🚀 주요 변경 내용

NoHandlerFoundException 설정 추가

  • 정의되지 않은 API에 접근 시 발생하는 NoHandlerFoundException 예외를 핸들링하도록 설정을 추가했습니다.
  • 해당 설정으로 통해 정적 리소스를 탐색하지 않습니다.
  • 관련해서 로그인 페이지를 지웠습니다.

💬 참고 사항


✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

@Soundbar91 Soundbar91 self-assigned this Mar 1, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7417fee and cf06fb1.

📒 Files selected for processing (1)
  • src/main/resources/application.yml
📜 Recent review details
🧰 Additional context used
📓 Path-based instructions (1)
**/*

⚙️ CodeRabbit configuration file

**/*: 공통 리뷰 톤 가이드:

  • 모든 코멘트는 첫 줄에 [LEVEL: ...] 태그를 포함한다.
  • 과장된 표현 없이 사실 기반으로 작성한다.
  • 한 코멘트에는 하나의 이슈만 다룬다.
  • 코드 예시가 필요하면 최소 수정 예시를 제시한다.
  • 가독성/단순화/확장성 이슈를 발견하면 우선순위를 높여 코멘트한다.

Files:

  • src/main/resources/application.yml
🔇 Additional comments (1)
src/main/resources/application.yml (1)

14-16: [LEVEL: INFO]

LGTM! 정적 리소스 매핑 비활성화 설정이 올바르게 추가되었습니다.

spring.web.resources.add-mappings: false 설정으로 정적 리소스 핸들러가 비활성화되어, 정의되지 않은 API 엔드포인트 접근 시 NoHandlerFoundException이 발생하도록 구성됩니다. 들여쓰기도 파일 내 다른 설정(예: servlet.multipart)과 동일한 2칸 증가 패턴(2→4→6)을 따르고 있습니다.


📝 Walkthrough

Walkthrough

정적 리소스 자동 매핑을 비활성화하는 설정이 추가되고, 프로젝트의 정적 로그인 페이지(src/main/resources/static/login.html)가 제거되었습니다. 기능적 변경은 정적 리소스 제공과 관련됩니다. (총 변경: 설정 1건 추가, 정적 파일 1건 삭제)

Changes

Cohort / File(s) Summary
설정 변경
src/main/resources/application.yml
spring.web.resources.add-mappings: false 추가 — Spring Boot의 자동 정적 리소스 매핑 비활성화.
정적 자원 제거
src/main/resources/static/login.html
한국어 기반의 정적 OAuth 로그인 페이지(google, naver, kakao, apple 옵션 포함) 파일 삭제. No server-side 코드 변경.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 분

🐰 설정 한 줄 무심히 놓고,
로그인 페이지는 살며시 보내네.
정적 길을 닫아두고,
다음 발걸음 준비하네.
깔끔한 변화, 토끼가 축하하네. 🥕✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 NoHandlerFoundException 설정 추가에 관한 내용이지만, 실제 변경사항은 spring.web.resources.add-mappings 설정 추가와 login.html 삭제입니다. 제목이 설정의 전체 목적을 명확히 반영하지 못합니다.
Description check ✅ Passed PR 설명은 NoHandlerFoundException 설정 추가, 정적 리소스 미탐색, 로그인 페이지 삭제에 관해 명확히 설명하고 있으며, 실제 변경사항과 관련이 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/not-found-resource

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/main/resources/application.yml`:
- Around line 14-16: The YAML block for the keys web, resources, and
add-mappings uses inconsistent indentation; update it so each nesting level uses
2 spaces like the rest of the file: place web: at the same base level as other
top-level keys, indent resources: by 2 spaces under web:, and indent
add-mappings: by 4 spaces under resources: (i.e., use the key names web,
resources, add-mappings to locate and correct the indentation).

ℹ️ Review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3be6936 and 7417fee.

📒 Files selected for processing (2)
  • src/main/resources/application.yml
  • src/main/resources/static/login.html
💤 Files with no reviewable changes (1)
  • src/main/resources/static/login.html
📜 Review details
🧰 Additional context used
📓 Path-based instructions (1)
**/*

⚙️ CodeRabbit configuration file

**/*: 공통 리뷰 톤 가이드:

  • 모든 코멘트는 첫 줄에 [LEVEL: ...] 태그를 포함한다.
  • 과장된 표현 없이 사실 기반으로 작성한다.
  • 한 코멘트에는 하나의 이슈만 다룬다.
  • 코드 예시가 필요하면 최소 수정 예시를 제시한다.
  • 가독성/단순화/확장성 이슈를 발견하면 우선순위를 높여 코멘트한다.

Files:

  • src/main/resources/application.yml
🔇 Additional comments (1)
src/main/resources/application.yml (1)

14-16: [LEVEL: INFO]

Spring Boot 3.5.8에서는 NoHandlerFoundException 설정이 기본값으로 활성화됩니다.

프로젝트는 Spring Boot 3.5.8을 사용하고 있으므로 spring.mvc.throw-exception-if-no-handler-found이 기본적으로 true입니다. 현재 구성된 web.resources.add-mappings: false와 함께 예외 처리가 올바르게 작동하므로 추가 설정이 불필요합니다.

Comment on lines +14 to +16
web:
resources:
add-mappings: false
Copy link

Choose a reason for hiding this comment

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

🧹 Nitpick | 🔵 Trivial

[LEVEL: MINOR]

YAML 들여쓰기 스타일이 파일 내 다른 설정과 일치하지 않습니다.

기존 파일은 레벨당 2칸 들여쓰기를 사용합니다 (config:import: = 2칸 → 4칸). 새로 추가된 설정은 web:resources:add-mappings:가 2칸 → 6칸 → 8칸으로 되어 있어 일관성이 없습니다.

♻️ 들여쓰기 수정 제안
   web:
-      resources:
-        add-mappings: false
+    resources:
+      add-mappings: false
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
web:
resources:
add-mappings: false
web:
resources:
add-mappings: false
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/main/resources/application.yml` around lines 14 - 16, The YAML block for
the keys web, resources, and add-mappings uses inconsistent indentation; update
it so each nesting level uses 2 spaces like the rest of the file: place web: at
the same base level as other top-level keys, indent resources: by 2 spaces under
web:, and indent add-mappings: by 4 spaces under resources: (i.e., use the key
names web, resources, add-mappings to locate and correct the indentation).

@Soundbar91 Soundbar91 merged commit 3480baf into develop Mar 1, 2026
2 checks passed
@Soundbar91 Soundbar91 deleted the feat/not-found-resource branch March 1, 2026 12:15
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