Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

supportsSingleSignOn removed from manifest when publishing #11980

Closed
ksdaniel opened this issue Jul 8, 2024 · 3 comments
Closed

supportsSingleSignOn removed from manifest when publishing #11980

ksdaniel opened this issue Jul 8, 2024 · 3 comments
Assignees
Labels
TA:Compute Team Area: Compute

Comments

@ksdaniel
Copy link

ksdaniel commented Jul 8, 2024

Describe the bug

When importing app packages that have the microsoftEntraConfiguration node in the authorization section - the microsoftEntraConfiguration gets removed during import.

This results in a validation error when we try to "preview" the application.

To Reproduce

Create a teams message with the following parameters:

  • message extension
  • api based
  • entra authentication

This will create a manifest file with the following node:

"authorization": {
                "authType": "microsoftEntra",
                "microsoftEntraConfiguration": {
                    "supportsSingleSignOn": true
                }
            }

Provision the resources for the application.

Import the application to Teams.

In the "imported" manifest the authorization node looks like this:

"authorization": {
"authType": "microsoftEntra"
},

When you try to preview the application you will get an validation message saying: Api based compose extension with microsoftEntra authType have no microsoftEntraConfiguration defined. Api based compose extension with microsoftEntra authType does not set supportsSingleSignOn to true

Expected behavior

After the application is deployed to teams we should see the authorization node in the manifest and the application should authenticate to the Azure Function using Entra.

VS Code Extension Information (please complete the following information):

Teams Toolkit v5.8.1
Microsoft Teams version 24152.415.2975.367.

Copy link
Contributor

Thank you for contacting us! Any issue or feedback from you is quite important to us. We will do our best to fully respond to your issue as soon as possible. Sometimes additional investigations may be needed, we will usually get back to you within 2 days by adding comments to this issue. Please stay tuned.

@microsoft-github-policy-service microsoft-github-policy-service bot added the needs attention This issue needs the attention of a contributor. label Jul 8, 2024
@huimiu huimiu added investigating needs attention This issue needs the attention of a contributor. and removed needs attention This issue needs the attention of a contributor. investigating labels Jul 9, 2024
@blackchoey blackchoey added investigating TA:Compute Team Area: Compute labels Jul 9, 2024
@huimiu
Copy link
Member

huimiu commented Jul 9, 2024

Hey @ksdaniel, thanks for reaching out. This is a known issue. The Teams Developer Platform doesn't currently support the Microsoft Entra for authentication.

But don't worry, you don't need to manually import your app into Teams. The Teams Toolkit will automatically sideload your app through a different method, not using the Teams Developer Platform. For instance, when you press F5 and the browser opens Teams, your message extension app should be there.

@huimiu huimiu removed investigating needs attention This issue needs the attention of a contributor. labels Jul 9, 2024
@huimiu
Copy link
Member

huimiu commented Aug 1, 2024

Hey @ksdaniel, let us close this issue first, feel free to let us know if you have any further questions, thanks!

@huimiu huimiu closed this as completed Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TA:Compute Team Area: Compute
Projects
None yet
Development

No branches or pull requests

3 participants