Skip to content

feat(runtime): add optional API token guard#916

Merged
Hmbown merged 1 commit into
mainfrom
integrate/pr-856-runtime-token
May 6, 2026
Merged

feat(runtime): add optional API token guard#916
Hmbown merged 1 commit into
mainfrom
integrate/pr-856-runtime-token

Conversation

@Hmbown
Copy link
Copy Markdown
Owner

@Hmbown Hmbown commented May 6, 2026

Integrates #856

Summary

  • add an opt-in --auth-token / DEEPSEEK_RUNTIME_TOKEN guard for runtime API /v1/* routes
  • keep default local behavior unchanged and leave /health public for readiness checks
  • document supported token transport for bearer headers, X-DeepSeek-Runtime-Token, and EventSource query-token clients

Test plan

  • cargo fmt --all -- --check
  • git diff --cached --check
  • cargo test -p deepseek-tui runtime_token_guard_protects_v1_routes --all-features --locked

Credit

Copilot AI review requested due to automatic review settings May 6, 2026 23:27
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Integrates #856 as a focused runtime API security slice. Default local behavior remains unchanged; /v1/* routes require a token only when --auth-token or DEEPSEEK_RUNTIME_TOKEN is set.

Co-authored-by: Zhuoran Deng <dengzhuoran9@gmail.com>
@Hmbown Hmbown force-pushed the integrate/pr-856-runtime-token branch from 72dee74 to 9406855 Compare May 6, 2026 23:28
@Hmbown Hmbown merged commit afe99f2 into main May 6, 2026
8 checks passed
@Hmbown Hmbown review requested due to automatic review settings May 6, 2026 23:52
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