[JS] fix: correct api key logic - Githubissues

microsoft / teams-ai

SDK focused on building AI based applications and extensions for Microsoft Teams and other Bot Framework channels

MIT License

349 stars 142 forks source link

[JS] fix: correct api key logic #1747

Closed swatDong closed 1 week ago

swatDong commented 2 weeks ago

Linked issues

closes: #1746

Details

Proposal to fix the wrong api-key header. Feel free to comment or create other fixes.

Change details

For OpenAI (apiKey), use "Authorization": "Bearer xxx"
For Azure OpenAI (azureApiKey), use "api-key": "xxx"
For Azure OpenAI (azureADTokenProvider), use "Authorization": "Bearer xxx"

Attestation Checklist

[x] My code follows the style guidelines of this project
I have checked for/fixed spelling, linting, and other errors
I have commented my code for clarity
I have made corresponding changes to the documentation (updating the doc strings in the code is sufficient)
My changes generate no new warnings
I have added tests that validates my changes, and provides sufficient test coverage. I have tested with:
- Local testing
- E2E testing in Teams
New and existing unit tests pass locally with my changes