microsoft / teams-ai

SDK focused on building AI based applications and extensions for Microsoft Teams and other Bot Framework channels
MIT License
359 stars 152 forks source link

[Feature Request]: TeamsAttachmendDownloader: handle custom tenant scenarios for public and gov #1561

Open corinagum opened 3 months ago

corinagum commented 3 months ago

Scenario

Right now, TeamsAttachmentDownloader handles getAccessToken for when public Azure and gov cloud do not have custom tenants or app credentials.

I consider this a P2 until someone chimes in with the need for this feature addition.

More information (only available to internal Teams/BF sync meeting attendees):

Solution

We should modify the code so that if MicrosoftAppCredentials or MicrosoftGovenmentAppCredentials have a tenant specified, the url is constructed as appropriate. I believe this means the tenant id is added into the url.

Additional Context

No response