Skip to content

Add CFAccess RoundTripper for prometheus tooling - #3

Open
jaymzh wants to merge 1 commit into
prometheus:mainfrom
jaymzh:initial-implementation
Open

jaymzh wants to merge 1 commit into
prometheus:mainfrom
jaymzh:initial-implementation

Conversation

@jaymzh

@jaymzh jaymzh commented Sep 24, 2026

Copy link
Copy Markdown

This introduces a small library that can be used in prometheus tooling
to enable CFAccess support. Primarily intended for CLI tooling, though
not limited to that.

This primarily provides two APIs:

  • PrepareHTTPClientConfig - detect and validate the cf-access
    configuration and return a copy with the authorization entry removed,
    preventing common from treating cf-access as a literal HTTP
    authorization scheme.
  • NewRoundTripper - provide a cf-access aware round-tripper

Included in this is tests and CI.

@jaymzh

jaymzh commented Sep 24, 2026 •

Copy link
Copy Markdown
Author

BTW I copied the basic CI structure from the sigv4 repo. Including the approval stuff, which I see now there's another PR for. If you just wanna use this PR, that's cool, if you wanna merge the other one, I'll rebase. As you wish.

@roidelapluie

Copy link
Copy Markdown
Member

Did not see your comment, can you please rebase? thanks :)

This introduces a small library that can be used in prometheus tooling
to enable CFAccess support. Primarily intended for CLI tooling, though
not limited to that.

This primarily provides two APIs:

* `PrepareHTTPClientConfig` - detect and validate the cf-access
  configuration and return a copy with the authorization entry removed,
  preventing common from treating cf-access as a literal HTTP
  authorization scheme.
* `NewRoundTripper` - provide a cf-access aware round-tripper

Included in this is tests and CI.

Signed-off-by: Phil Dibowitz <phil@ipom.com>
@jaymzh
jaymzh force-pushed the initial-implementation branch from 4a722b7 to 766ae74 Compare September 25, 2026 14:13
@jaymzh

jaymzh commented Sep 25, 2026

Copy link
Copy Markdown
Author

No problem @roidelapluie , done!

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.

2 participants