Merge pull request #1582 from AzureAD/cloud-discovery

Use instance discovery for trusted hosts and combine authority classes

Author: tnorling

lib/msal-angularjs/package-lock.json

@@ -44,6 +44,7 @@ import { Constants,
     FramePrefix
 } from "./utils/Constants";
 import { CryptoUtils } from "./utils/CryptoUtils";
+import { TrustedAuthority } from "./authority/TrustedAuthority";
 
 // default authority
 const DEFAULT_AUTHORITY = "https://login.microsoftonline.com/common";
@@ -219,7 +220,7 @@ export class UserAgentApplication {
 
         this.telemetryManager = this.getTelemetryManagerFromConfig(this.config.system.telemetry, this.clientId);
 
-        AuthorityFactory.setKnownAuthorities(this.config.auth.validateAuthority, this.config.auth.knownAuthorities);
+        TrustedAuthority.setTrustedAuthoritiesFromConfig(this.config.auth.validateAuthority, this.config.auth.knownAuthorities);
         AuthorityFactory.saveMetadataFromConfig(this.config.auth.authority, this.config.auth.authorityMetadata);
 
         // if no authority is passed, set the default: "https://login.microsoftonline.com/common"

lib/msal-core/src/UserAgentApplication.ts

@@ -3,6 +3,8 @@
  * Licensed under the MIT License.
  */
 
+import { NetworkRequestType } from "./utils/Constants";
+
 /**
  * XHR client for JSON endpoints
  * https://www.npmjs.com/package/async-promise
@@ -43,7 +45,7 @@ export class XhrClient {
                 reject(xhr.status);
             };
 
-            if (method === "GET") {
+            if (method === NetworkRequestType.GET) {
                 xhr.send();
             }
             else {