ManageIQ / azure-armrest

Ruby interface for Azure using the new REST API
Apache License 2.0
15 stars 36 forks source link

Change the resource URL to the active directory URL #365

Closed djberg96 closed 6 years ago

djberg96 commented 6 years ago

It would seem that for initial authentication and token generation, we should be using the ActiveDirectory URL instead of the resource manager URL to get a token.

https://docs.microsoft.com/en-us/azure/active-directory/develop/active-directory-protocols-oauth-code

Normally this doesn't matter when authenticating against Microsoft Azure, but it comes into play when using Azure Stack.

miq-bot commented 6 years ago

Checked commit https://github.com/djberg96/azure-armrest/commit/4c6509a816a9e16623ad2e840392606974a3ab74 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 1 file checked, 0 offenses detected Everything looks fine. :star: