Skip to content

Protect AI and tracing routes with API key#3399

Open
MaxGhenis wants to merge 2 commits intomasterfrom
codex/protect-ai-routes
Open

Protect AI and tracing routes with API key#3399
MaxGhenis wants to merge 2 commits intomasterfrom
codex/protect-ai-routes

Conversation

@MaxGhenis
Copy link
Copy Markdown
Collaborator

Summary

  • extend the shared API-key guard to the AI prompt and tracer-analysis routes in addition to simulation-analysis
  • add focused route tests for unauthorized and authorized access to the protected AI routes
  • add a changelog fragment for the new protection

Testing

  • FLASK_DEBUG=1 uv run --python 3.11 pytest tests/unit/routes/test_ai_route_auth.py tests/unit/routes/test_simulation_analysis_auth.py -q

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.48%. Comparing base (ddd65d8) to head (5c91913).
⚠️ Report is 10 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3399      +/-   ##
==========================================
+ Coverage   72.58%   73.48%   +0.90%     
==========================================
  Files          56       57       +1     
  Lines        2477     2614     +137     
  Branches      431      466      +35     
==========================================
+ Hits         1798     1921     +123     
- Misses        598      608      +10     
- Partials       81       85       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MaxGhenis MaxGhenis force-pushed the codex/protect-ai-routes branch 2 times, most recently from 34261b0 to 7dedfcd Compare April 12, 2026 19:28
@MaxGhenis MaxGhenis force-pushed the codex/protect-ai-routes branch from 7dedfcd to 5c91913 Compare April 12, 2026 19:34
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