microsoft / teams-ai

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

[PY] [Bug]: signinVerify invoke may be processed by wrong OAuth Connection #1624

Closed singhk97 closed 22 hours ago

singhk97 commented 4 months ago

816

skoryky commented 2 months ago

I'm running into an issue where the signinVerify invoke isn't being handled. I wasn't able to find the Python equivalent to the JS library's BotAuthentication where the handler is registered. Where is it done in the Python library?

Is the functionality instead meant to be called directly in AuthManager? If so, this code isn't being called because the function returns early here.

It's simple enough to fix there, but then ultimately a 501 Not Implemented is returned on the signinVerify invoke call because presumably there is no registered handler.

corinagum commented 22 hours ago

Closing as duplicate