No option to use OAuth with sandbox accounts

[hassanakbar4]

These URLs are for production accounts.

      SCOPE = "https://ads.microsoft.com/msads.manage"
      AUTHORIZE_URI = "https://login.microsoftonline.com/common/oauth2/v2.0/authorize"
      TOKEN_URI = "https://login.microsoftonline.com/common/oauth2/v2.0/token"
      REDIRECT_URI = "https://login.microsoftonline.com/common/oauth2/nativeclient"

Instead of these, we need to add sandbox URLs based on :environment.

[danielmbarlow]

Thanks for pointing that out. I was just reading authorization_handler.rb.

Just an idea, but maybe we could achieve that as follows:

• Adding a parameter to the constructor (environment: :production for example), and saving to @environment
• Adding a hash constant to AuthorizationHandler like this:

API_URLS = {
  production: {
    scope: "...."
    ... etc ...
  }
  sandbox: {
    ...
  }
}

• Adding private methods for #scope, #authorize_url etc. to switch on @environment, maybe using API_URLS.dig(@environment, :scope) for example
• Using those private methods wherever we used the constants before

Is anyone keen to submit a Pull Request for that feature? @hassanakbar4 maybe? Perhaps you could test your branch with your app and let us know how it goes