🛡️ Sentinel: [CRITICAL] 비 ASCII API 키로 인한 500 서버 오류(DoS) 취약점 수정

[seonghobae]

🚨 Severity: CRITICAL
💡 Vulnerability: hmac.compare_digest 함수는 비 ASCII 문자가 포함된 문자열을 비교할 때 TypeError를 발생시킵니다. 공격자가 비정상적인 비 ASCII 문자가 포함된 X-API-Key 헤더를 전송하면, 앱이 401 응답 대신 500 서버 오류로 붕괴될 수 있습니다.
🎯 Impact: 이를 악용하면 간단한 요청만으로 서버에 처리되지 않은 예외를 반복적으로 발생시켜 서비스 거부(DoS) 상태를 유발할 수 있습니다.
🔧 Fix: hmac.compare_digest 호출 시 제공된 키와 설정된 키 모두 .encode("utf-8")을 사용하여 명시적으로 바이트로 변환한 후 비교하도록 수정했습니다.
✅ Verification: tests/test_saas_web.py에 비 ASCII 문자가 포함된 API 키 헤더를 전송해도 500 오류 없이 401 응답이 정상 반환되는지 확인하는 단위 테스트를 추가하고 통과를 확인했습니다.

---

PR created automatically by Jules for task 3473964550003763815 started by @seonghobae

Summary by CodeRabbit

• 버그 수정

  • 비ASCII 문자가 포함된 API 키도 인증 처리 중 오류 없이 안전하게 검증되도록 개선했습니다.
  • 잘못된 API 키 입력 시 서버 오류 대신 HTTP 401 응답이 반환됩니다.

• 테스트

  • 비ASCII API 키에 대한 인증 동작을 검증하는 테스트를 추가했습니다.

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d638cb12-40fe-4633-a58e-37ae90e03359

📥 Commits

Reviewing files that changed from the base of the PR and between 5a92586 and 03c61e9.

📒 Files selected for processing (3)

• .jules/sentinel.md
• saas_web.py
• tests/test_saas_web.py

---

📝 Walkthrough

Walkthrough

API 키 비교 전에 설정값과 요청 헤더를 UTF-8 바이트로 변환합니다. 비ASCII API 키 입력에서 예외가 발생하지 않고 401 응답이 반환되는 회귀 테스트와 Sentinel 기록을 추가했습니다.

Changes

API 키 인증 안정성

Layer / File(s)	Summary
API 키 비교 및 회귀 검증 saas_web.py, tests/test_saas_web.py, .jules/sentinel.md	hmac.compare_digest에 UTF-8 바이트를 전달하도록 변경했습니다. 비ASCII X-API-Key 헤더가 401 응답을 반환하는지 테스트합니다. 동일한 취약점과 대응책을 Sentinel에 기록했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• ContextualWisdomLab/codec-carver#303: 동일한 API 키 비교 수정과 비ASCII 입력에 대한 DoS 대응을 다룹니다.
• ContextualWisdomLab/codec-carver#308: saas_web.py의 API 키 비교 수정과 유니코드 인증 테스트를 포함합니다.
• ContextualWisdomLab/codec-carver#326: 비교 전에 API 키를 UTF-8 바이트로 변환하고 회귀 테스트를 추가합니다.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 비ASCII API 키로 발생하는 500 오류와 DoS 취약점 수정이라는 핵심 변경을 명확하게 설명합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch sentinel-fix-hmac-dos-3473964550003763815

---

_{Comment @coderabbitai help to get the list of available commands.}

[seonghobae]

Superseded by canonical PR #361, which includes this non-ASCII API-key rejection and the larger credential-registry and result-retention security boundary. Closing the duplicate so validation converges on one head.

[google-labs-jules]

Superseded by canonical PR #361, which includes this non-ASCII API-key rejection and the larger credential-registry and result-retention security boundary. Closing the duplicate so validation converges on one head.

Understood. Acknowledging that this work is superseded by PR #361 and is now obsolete. Stopping work on this task.