Skip to content

Conversation

@filiphr
Copy link
Contributor

@filiphr filiphr commented May 30, 2025

This is a continuation of #3364. It makes sure that the ApiKey is resolved as late as possible

Signed-off-by: Filip Hrisafov <[email protected]>
@filiphr filiphr force-pushed the open-ai-dynamic-api-key branch from b86228b to 08510d6 Compare May 30, 2025 12:32
@ilayaperumalg ilayaperumalg self-assigned this May 30, 2025
@ilayaperumalg
Copy link
Member

@filiphr Thanks for the follow up PR!

@filiphr filiphr force-pushed the open-ai-dynamic-api-key branch from 4dbe8c9 to a48e608 Compare May 31, 2025 14:32
@ilayaperumalg
Copy link
Member

ilayaperumalg commented Jun 2, 2025

@filiphr Thanks for following it up with the fix to check for additional http headers (if available). The changes look good. Squashed, rebased, updated the author info and merged as 64ea88a

@filiphr
Copy link
Contributor Author

filiphr commented Jun 2, 2025

Thanks @ilayaperumalg. Should we close this PR now?

@ilayaperumalg
Copy link
Member

Yes, I will close this PR as soon as the merge build is complete along with the back-port to 1.0.x branch

@ilayaperumalg
Copy link
Member

The latest build was successful! Back-ported into 1.0.x as dd6c0a9

@filiphr filiphr deleted the open-ai-dynamic-api-key branch June 2, 2025 17:22
@filiphr
Copy link
Contributor Author

filiphr commented Jun 2, 2025

Thanks @ilayaperumalg. I was not sure about the process, but all good 😄

@ilayaperumalg
Copy link
Member

No problem @filiphr. Thank you for your contributions and looking forward to see more!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants