lua-openai icon indicating copy to clipboard operation
lua-openai copied to clipboard

Support additional headers

Open johnd0e opened this issue 1 year ago • 4 comments

johnd0e avatar Apr 05 '24 00:04 johnd0e

Could you explain more about why you want this, and why it should be something stored on the client instance?

Thanks

leafo avatar Apr 05 '24 01:04 leafo

Sure. For example OpenAI-Organization. And there are some 3rd-party services, that are not exactly openai-compatible, and require extra headers to pass additional info. E.g. see https://docs.helicone.ai/getting-started/integration-method/openai-proxy

johnd0e avatar Apr 05 '24 08:04 johnd0e

Thanks for the contribution. My only suggestion is that the more_headers argument takes precedence over the instance level headers object, meaning it's merged after the headers object

leafo avatar Apr 17 '24 00:04 leafo

My only suggestion is [...]

Sure. Done

johnd0e avatar Apr 17 '24 15:04 johnd0e