-
Notifications
You must be signed in to change notification settings - Fork 145
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
Add Schema Compare support for AAD #853
Conversation
(weird issue not letting me add comment in code) SchemacompareUtils.cs line 121 : |
(also not letting me edit the comment :( ) One more thing, I noticed one SC test failed in PR build... please confirm locally that all SC tests are passing with the change |
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.
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.
@udeeshagautam added null check and the test passed when I ran it locally with the changes. |
src/Microsoft.SqlTools.ServiceLayer/SchemaCompare/SchemaCompareUtils.cs
Outdated
Show resolved
Hide resolved
…to CreateSchemaCompareEndpoint()
Adds support for AAD in schema compare by passing the AzureAcountToken if there is one. Fixes microsoft/azuredatastudio#5123