fix: validate_startup checks CORS wildcard in production #479

Closed
kimi wants to merge 0 commits from kimi/issue-472 into main
Collaborator

Summary

  • validate_startup() now rejects wildcard * in CORS_ORIGINS when running in production mode
  • Adds test to verify the new check triggers sys.exit(1)

Fixes #472

## Summary - `validate_startup()` now rejects wildcard `*` in `CORS_ORIGINS` when running in production mode - Adds test to verify the new check triggers `sys.exit(1)` Fixes #472
kimi added 1 commit 2026-03-19 19:31:56 +00:00
fix: validate_startup checks CORS wildcard in production
All checks were successful
Tests / lint (pull_request) Successful in 4s
Tests / test (pull_request) Successful in 1m4s
063afe2573
validate_startup() now exits with an error if CORS_ORIGINS contains
a wildcard '*' in production mode, matching the runtime stripping
already done in _get_cors_origins().

Fixes #472

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Owner

Closing — the CORS wildcard production check was already implemented and merged in PR #478. This PR is redundant. The only difference is the error message wording. #472 is already closed.

Closing — the CORS wildcard production check was already implemented and merged in PR #478. This PR is redundant. The only difference is the error message wording. #472 is already closed.
Timmy closed this pull request 2026-03-19 19:34:40 +00:00

Pull request closed

Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Rockachopa/Timmy-time-dashboard#479