Skip to content

Add custom headers and base address customization - #27

Open
timabdulla wants to merge 1 commit into
gbaptista:mainfrom
sanjustoresearch:add-request-customization
Open

Add custom headers and base address customization#27
timabdulla wants to merge 1 commit into
gbaptista:mainfrom
sanjustoresearch:add-request-customization

Conversation

@timabdulla

@timabdulla timabdulla commented Sep 13, 2024

Copy link
Copy Markdown

Hi,

This is a simple PR to add the ability to customize the base address (using options[:credentials][:base_address]) and to provide custom headers (using options[:headers]). This is quite useful for integrating with proxy services.

Thanks,

Tim

@JerakRus

Copy link
Copy Markdown

Hi, thanks for the work you've done! I tested your functionality, and it works great!

However, in my case, I used my own proxy server on a VPS, and since my server is just a proxy without a domain (only an IP), I needed more flexibility in configuring SSL certificates for Faraday.

I forked your fork and improved the handling of parameters in connection: { ssl: {...} }. These changes wouldn't make sense without your request, so I could submit a PR to your fork if you're interested, or directly here after your branch is approved.

My fork can be viewed here: JerakRus@9a5a952

@stevenou

stevenou commented May 2, 2025

Copy link
Copy Markdown
Contributor

Curious what's happening to this PR or related functionality? I could use custom headers in order to enable control over provisioned throughput usage

@stevenou

Copy link
Copy Markdown
Contributor

@gbaptista any thoughts on this PR?

@kapso

kapso commented Mar 14, 2026

Copy link
Copy Markdown

@gbaptista Google is increasingly recommending Priority Pay Go Headers to get around 429 errors. Can we merge this PR?

https://docs.cloud.google.com/vertex-ai/generative-ai/docs/priority-paygo#initialize-use-priority

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.

4 participants