A detailed description of the problem can be found in the linked issue. The solution - in simple terms - was to add the settingName property in the ITurnContext.Activity.Value property so that the route selectors for signin/verifyState and signin/tokenExchange will trigger the correct authentication setting.
Change details
Added methods to get/set setting name in the ITurnContext.Activity.Value.
Updated TokenExchangeRouteSelector & VerifyStateRouteSelector to check for the settingName in the context object.
Implemented unit tests
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
Linked issues
closes: #1621 (issue number)
Details
A detailed description of the problem can be found in the linked issue. The solution - in simple terms - was to add the
settingName
property in theITurnContext.Activity.Value
property so that the route selectors forsignin/verifyState
andsignin/tokenExchange
will trigger the correct authentication setting.Change details
ITurnContext.Activity.Value
.TokenExchangeRouteSelector
&VerifyStateRouteSelector
to check for thesettingName
in the context object.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:
New and existing unit tests pass locally with my changes