Updated AzureCloud endpoint; Added support for AzureUSGov & AzureGermany#3182
Updated AzureCloud endpoint; Added support for AzureUSGov & AzureGermany#3182shahabhijeet merged 3 commits intoAzure:vs17Devfrom
Conversation
|
@horrion, It will cover your contributions to all Microsoft-managed open source projects. |
|
@horrion, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request. |
| private static readonly ActiveDirectoryServiceSettings AzureSettings = new ActiveDirectoryServiceSettings | ||
| { | ||
| AuthenticationEndpoint= new Uri("https://login.windows.net/"), | ||
| AuthenticationEndpoint= new Uri("https://login.microsoftonline.com/"), |
There was a problem hiding this comment.
@horrion any reason this is needed? What is causing your make this change? Would like to understand?
| ValidateAuthority = true | ||
| }; | ||
|
|
||
| private static readonly ActiveDirectoryServiceSettings AzureUSGovernmentSettings = new ActiveDirectoryServiceSettings |
There was a problem hiding this comment.
@horrion
Have you tested if these changes are required and will work?
Where are the tests for this newly added properties?
There was a problem hiding this comment.
How would you like me to test these changes?
There was a problem hiding this comment.
@horrion Apart from the endpoints (and thank you for adding the ARM endpoint), I am more interested to know if these endpoints do work for authenticating against various cloud environments? And hence wanted to know if you have used the piece of code that you have contributed?
Or was your intention to simply add those endpoints?
Because majority of our Authentication scenarios are via our TestFramework when automating our tests
https://github.com/Azure/azure-sdk-for-net/blob/vs17Dev/src/SdkCommon/TestFramework/ClientRuntime.Azure.TestFramework/TestEnvironment.cs#L254
Description
View this for further information. This is the Pull Request for the vs17Dev branch.
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines
Testing Guidelines
SDK Generation Guidelines
project.jsonandAssemblyInfo.csfiles have been updated with the new version of the SDK.