Skip to content

feat(client): add custom auth header name support#75

Merged
cjumel merged 3 commits into
LinkupPlatform:mainfrom
EstebanVincent:feat/custom_base_url
May 29, 2026
Merged

feat(client): add custom auth header name support#75
cjumel merged 3 commits into
LinkupPlatform:mainfrom
EstebanVincent:feat/custom_base_url

Conversation

@EstebanVincent
Copy link
Copy Markdown
Contributor

@EstebanVincent EstebanVincent commented May 28, 2026

Description

Some API gateways (e.g. Azure APIM) require the key in a
non-standard header like Ocp-Apim-Subscription-Key instead of
Authorization: Bearer. auth_header lets callers override the
header name; the key value is sent as-is when set.

Checklist

  • I have installed prek on this project (for instance with the make install-dev command)
    before creating any commit, or I have run successfully the make lint command on my changes.
  • I have run successfully the make typecheck test command on my changes.
  • I have updated the README.md if my changes affected it.

Some API gateways (e.g. Azure APIM) require the key in a
non-standard header like `Ocp-Apim-Subscription-Key` instead of
`Authorization: Bearer`. `auth_header` lets callers override the
header name; the key value is sent as-is when set.
@EstebanVincent EstebanVincent changed the title Feat/custom base url Feat/custom base header key May 28, 2026
@cjumel cjumel changed the title Feat/custom base header key feat(client): add custom auth header name support May 29, 2026
Copy link
Copy Markdown
Contributor

@cjumel cjumel left a comment

Choose a reason for hiding this comment

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

Hello! Thanks for the contribution. Just one very small detail to address. 🙂

Comment thread tests/unit/client_test.py Outdated
@cjumel cjumel merged commit 434751e into LinkupPlatform:main May 29, 2026
3 checks passed
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