coder / coder/internal

AI Bridge Proxy: authenticate tunneled traffic

Open
#1,352 1 comment 0 reactions 1 assignee Assigned to @evgeniy-scherbina View on GitHub
must-do
Dominant language
No language data
Stars
3
Forks
0
PR merge metrics
No merged PRs in 30d

Description

## Problem

AI Bridge Proxy does not perform any authentication on tunneled traffic. This applies to:
* Non-allowlisted HTTPS CONNECT requests: tunneled via `tunneledMiddleware` without any credential check
* Plain HTTP proxy requests: forwarded without any credential check (related to bug https://github.com/coder/internal/issues/1351)

For allowlisted domains (AI providers), the proxy extracts the Coder session token from `Proxy-Authorization` and forwards it to AI Bridge, which validates it. However, for tunneled traffic there is no AI Bridge in the path, so no authentication happens at all.

This means anyone who can reach the proxy can tunnel traffic through it to any non-allowlisted destination without credentials. This is a known limitation documented in [Security Considerations](https://coder.com/docs/ai-coder/ai-bridge/ai-bridge-proxy/setup#security-considerations).

## Proposal

Validate authentication for all tunneled traffic before forwarding. The proxy should check the `Proxy-Authorization` header on both non-allowlisted CONNECT requests and plain HTTP proxy requests.

## Open Questions

The proxy currently does not validate tokens itself, for allowlisted domains, it passes the token to AI Bridge which validates it against Coder. For tunneled traffic there is no AI Bridge in the path, so the proxy would need to validate the token directly.

### Options:

* Check header exists only: verify that `Proxy-Authorization` is present but don't validate the token. Simple but weak, any string would pass.
* Validate against Coder API: make a request to Coder to verify the token. Correct but expensive, every tunneled request would hit the Coder API.
* Validate with caching: validate the token against Coder API on first use and cache the result for a period. Balances correctness with performance.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.