Skip to content

feat: refactor provider clients to inject default app headers automat…#235

Merged
appleboy merged 1 commit intomainfrom
transport
May 1, 2025
Merged

feat: refactor provider clients to inject default app headers automat…#235
appleboy merged 1 commit intomainfrom
transport

Conversation

@appleboy
Copy link
Owner

@appleboy appleboy commented May 1, 2025

…ically

  • Move DefaultHeaderTransport and NewHeaders utilities into a shared core/transport package
  • Add automatic injection of x-app-name and x-app-version headers to outgoing HTTP requests using values from the version package
  • Inject default headers in provider clients (Anthropic, Gemini, OpenAI) by wrapping HTTP client transports with DefaultHeaderTransport
  • Remove duplicated DefaultHeaderTransport and NewHeaders implementations from provider code
  • Set and use the App field in the version package for branding in headers

…ically

- Move DefaultHeaderTransport and NewHeaders utilities into a shared core/transport package
- Add automatic injection of x-app-name and x-app-version headers to outgoing HTTP requests using values from the version package
- Inject default headers in provider clients (Anthropic, Gemini, OpenAI) by wrapping HTTP client transports with DefaultHeaderTransport
- Remove duplicated DefaultHeaderTransport and NewHeaders implementations from provider code
- Set and use the App field in the version package for branding in headers

Signed-off-by: appleboy <[email protected]>
@appleboy appleboy merged commit c4d6669 into main May 1, 2025
10 checks passed
@appleboy appleboy deleted the transport branch May 1, 2025 09:22
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.

1 participant