Skip to content

Update Connect-MicrosoftTeams.md #8287

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

Merged
merged 2 commits into from
Sep 9, 2021

Conversation

alexdale0
Copy link
Contributor

Add a warning that basic auth is needed to run legacy *-Cs cmdlets.
@opbld31
Copy link

opbld31 commented Sep 2, 2021

Docs Build status updates of commit ad70cb0:

✅ Validation status: passed

File Status Preview URL Details
teams/teams-ps/teams/Connect-MicrosoftTeams.md ✅Succeeded View (teams-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@mark3grahams
Copy link

mark3grahams commented Sep 2, 2021

Hi alexdale0,

I do not think this is a basic auth issue. It works fine with user creds. This error presents when using an app registration.

Can you get this working? If so, how?

@alexdale0
Copy link
Contributor Author

Hi @mark3grahams - that's what this warning added to the docs is also saying. You must use basic auth/user creds, as authentication with Application ID+Certificate is not supported currently.

@mark3grahams
Copy link

So, it is flat-out broken? That's what I thought. Is there an ETA as to when this will be fixed? Or is there some other method whereby a script can run without a user account?

@alexdale0
Copy link
Contributor Author

So, it is flat-out broken? That's what I thought. Is there an ETA as to when this will be fixed? Or is there some other method whereby a script can run without a user account?

Unfortunately I don't have an ETA at this time. Will be sure to get these docs updated further when that changes though.

@yogkumgit yogkumgit requested review from get-itips, dariomws and a user September 6, 2021 15:01
Copy link
Contributor

@dariomws dariomws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alexdale0 Did you realize that the link you added is related to Exchange Online? Thanks!

@yogkumgit
Copy link

@tseward

@alexdale0
Copy link
Contributor Author

Hi @dariomws - I did, as it has some good info related to the warning message. If that's not a good practice let me know and I can see if I can find a reference somewhere else though.

Copy link
Contributor

@dariomws dariomws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alexdale0,

I cannot clearly see how it's related but I'm approving it because it could be a personal interpretation.

Thanks!

@tiburd tiburd assigned ghost Sep 7, 2021
@yogkumgit
Copy link

@tiburd Please merge this PR. Thanks

@opbld31
Copy link

opbld31 commented Sep 9, 2021

Docs Build status updates of commit 3eafd45:

✅ Validation status: passed

File Status Preview URL Details
teams/teams-ps/teams/Connect-MicrosoftTeams.md ✅Succeeded View (teams-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@tiburd tiburd merged commit f659ec7 into MicrosoftDocs:master Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants