Skip to content

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

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

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.

This branch has not been deployed

No deployments
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