Skip to content

feat(aisix): add config.proxy.expose_debug_headers - #415

Closed
jarvis9443 wants to merge 1 commit into
mainfrom
feat/aisix-expose-debug-headers-1917
Closed

jarvis9443 wants to merge 1 commit into
mainfrom
feat/aisix-expose-debug-headers-1917

Conversation

@jarvis9443

Copy link
Copy Markdown
Contributor

Adds config.proxy.expose_debug_headers (default false) to the aisix chart. It is rendered into the gateway config file through the ConfigMap like the other config.proxy keys. The gateway key comes from api7/aisix#1364: it turns on the x-aisix-call-id / x-aisix-served-by / x-aisix-route / x-aisix-cache* response headers on /v1/chat/completions, which are off by default from that release on.

This PR is held for the chart bump of the release that ships api7/aisix#1364. A gateway image older than that rejects the unknown key, so don't merge it before then. The chart-vs-gateway config drift check fails until appVersion points at that release.

Related issue: api7/AISIX-Cloud#1917

🤖 Generated with Claude Code

@jarvis9443 jarvis9443 added the aisix-hold-for-release Gateway-key chart change: merges with the release that ships the key label Oct 28, 2026
@coderabbitai

coderabbitai Bot commented Oct 28, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@jarvis9443 jarvis9443 mentioned this pull request Oct 29, 2026
@jarvis9443

Copy link
Copy Markdown
Contributor Author

Folded into the 1.8.0 release PR #418, which carries this commit.

@jarvis9443 jarvis9443 closed this Oct 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

aisix-hold-for-release Gateway-key chart change: merges with the release that ships the key

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant