Skip to content

feat: conditionally send aibridge actor headers#21643

Merged
dannykopping merged 7 commits into
mainfrom
dk/aibridge-upstream-headers
Jan 26, 2026
Merged

feat: conditionally send aibridge actor headers#21643
dannykopping merged 7 commits into
mainfrom
dk/aibridge-upstream-headers

Conversation

@dannykopping
Copy link
Copy Markdown
Contributor

@dannykopping dannykopping commented Jan 23, 2026

Also passes along the authenticated username as actor metadata.

Closes coder/aibridge#135
Depends on coder/aibridge#142

Replace aibridge tag with merge commit once coder/aibridge#142 lands.

@dannykopping dannykopping changed the title chore: add send actor header flag feat: conditionally send aibridge actor headers Jan 23, 2026
@dannykopping dannykopping changed the title feat: conditionally send aibridge actor headers feat: conditionally send aibridge actor headers Jan 23, 2026
@dannykopping dannykopping marked this pull request as ready for review January 23, 2026 09:37
Copy link
Copy Markdown
Contributor

@ssncferreira ssncferreira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Comment thread enterprise/aibridged/http.go
Comment thread codersdk/deployment.go
Comment thread enterprise/aibridged/http.go
Signed-off-by: Danny Kopping <danny@coder.com>
Signed-off-by: Danny Kopping <danny@coder.com>
Signed-off-by: Danny Kopping <danny@coder.com>
Signed-off-by: Danny Kopping <danny@coder.com>
Signed-off-by: Danny Kopping <danny@coder.com>
@dannykopping dannykopping force-pushed the dk/aibridge-upstream-headers branch from bc12bb4 to 98ac4df Compare January 26, 2026 14:31
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 26, 2026

Signed-off-by: Danny Kopping <danny@coder.com>
@dannykopping dannykopping enabled auto-merge (squash) January 26, 2026 14:54
@coder-tasks
Copy link
Copy Markdown
Contributor

coder-tasks Bot commented Jan 26, 2026

Documentation Check

No Changes Needed

The new --aibridge-send-actor-headers flag has comprehensive documentation in the auto-generated CLI reference (docs/reference/cli/server.md). The flag description clearly explains:

  • Purpose: Sends actor identification headers to upstream AI providers
  • When to use: Only needed when using a proxy between AI Bridge and upstream providers
  • Headers sent: X-Ai-Bridge-Actor-Id (user ID) and X-Ai-Bridge-Actor-Metadata-Username (username)
  • Default: false (opt-in behavior)

This is a low-level configuration flag with a narrow, well-documented use case. The existing AI Bridge admin guides (docs/ai-coder/ai-bridge/setup.md) already mention using proxies with base URL configuration for similar scenarios, so no additional admin documentation is needed.


Automated review via Coder Tasks

Signed-off-by: Danny Kopping <danny@coder.com>
@dannykopping dannykopping merged commit 7123518 into main Jan 26, 2026
31 checks passed
@dannykopping dannykopping deleted the dk/aibridge-upstream-headers branch January 26, 2026 15:08
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Send initiator identity and other configured metadata with upstream requests

2 participants