Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[extension/bearerauthtoken] Custom auth header support for bearertokenauth #36986

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

omleteer
Copy link

@omleteer omleteer commented Dec 30, 2024

Adding a config param for bearertokenauth to use a http header other than "Authorization" to read the token param from

Description

Today the bearertokenauth (BTA) by default fetches the auth token from the http "Authorization" header. By allowing multiple auth headers this enables easier attribution as well as decoupling of an extension from a pipeline.

Link to tracking issue

Fixes #36985

Testing

  1. Default config creation ensures that Header value if not specified is set to "Authorization" (defaultHeader)
  2. Ensured current unit tests work are working as intended.
  3. Added extra test to validate behavior with this parameter set (and to ensure "defaultheader" fails)

Documentation

Please see the README.md update.

@omleteer omleteer requested review from jpkrohling and a team as code owners December 30, 2024 23:55
@github-actions github-actions bot requested a review from frzifus December 30, 2024 23:56
@omleteer omleteer changed the title Custom auth header support for bearertokenauth [extension/bearerauthtoken] Custom auth header support for bearertokenauth Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow custom http headers for bearertokenauth
2 participants