-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Update Connect-MicrosoftTeams.md #8287
Conversation
Add a warning that basic auth is needed to run legacy *-Cs cmdlets.
Docs Build status updates of commit ad70cb0: ✅ Validation status: passed
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:
|
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? |
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. |
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. |
There was a problem hiding this 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!
@tseward |
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cannot clearly see how it's related but I'm approving it because it could be a personal interpretation.
Thanks!
@tiburd Please merge this PR. Thanks |
Docs Build status updates of commit 3eafd45: ✅ Validation status: passed
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:
|
Add a warning that basic auth is needed to run legacy *-Cs cmdlets. Related to https://github.com/MicrosoftDocs/office-docs-powershell/issues/7518 and https://github.com/MicrosoftDocs/office-docs-powershell/issues/7464 and https://github.com/MicrosoftDocs/office-docs-powershell/issues/7824