@azure/identity 2.0.3-alpha.20220211.2 → 2.0.3

package/CHANGELOG.md

@@ -1,28 +1,16 @@
 # Release History
 
-## 2.1.0-beta.1 (Unreleased)
+## 2.0.3 (2022-02-16)
 
 ### Features Added
 
-- Added log warning for non-support of user assigned identity in Managed Identity credentials in Cloud Shell environments
-
-### Breaking Changes
+- Added log warning for non-support of user assigned identity in Managed Identity credentials in Cloud Shell environments.
 
 ### Bugs Fixed
 
-### Other Changes
-
-## 2.0.3 (Unreleased)
-
-### Features Added
-
-- Added log warning for non-support of user assigned identity in Managed Identity credentials in Cloud Shell environments
-
-### Breaking Changes
-
-### Bugs Fixed
-
-### Other Changes
+- Fixed bug that duplicated the tenant Id on the URI of outgoing requests when passing an `authorityHost` ending with a tenant Id.
+- `ManagedIdentityCredential` now won't retry when it tries to ping the IMDS endpoint.
+- Now we are specifying the maximum number of retries to 3 to ensure that maximum retries won't change without notice.
 
 ## 2.0.2 (2022-02-03)
 

package/dist/index.js

@@ -123,7 +123,7 @@ class AuthenticationError extends Error {
                 errorDescription: "An unknown error occurred and no additional details are available.",
             };
         }
-        super(`${errorResponse.error}(status code ${statusCode}).\nMore details:\n${errorResponse.errorDescription}`);
+        super(`${errorResponse.error} Status code: ${statusCode}\nMore details:\n${errorResponse.errorDescription}`);
         this.statusCode = statusCode;
         this.errorResponse = errorResponse;
         // Ensure that this type reports the correct name
@@ -376,7 +376,9 @@ class IdentityClient extends coreClient.ServiceClient {
         if (!baseUri.startsWith("https:")) {
             throw new Error("The authorityHost address must use the 'https' protocol.");
         }
-        super(Object.assign(Object.assign({ requestContentType: "application/json; charset=utf-8" }, options), { userAgentOptions: {
+        super(Object.assign(Object.assign({ requestContentType: "application/json; charset=utf-8", retryOptions: {
+                maxRetries: 3,
+            } }, options), { userAgentOptions: {
                 userAgentPrefix,
             }, baseUri }));
         this.authorityHost = baseUri;
@@ -601,6 +603,9 @@ function getAuthority(tenantId, host) {
     if (!host) {
         host = DefaultAuthorityHost;
     }
+    if (new RegExp(`${tenantId}/?$`).test(host)) {
+        return host;
+    }
     if (host.endsWith("/")) {
         return host + tenantId;
     }
@@ -2769,15 +2774,20 @@ class ManagedIdentityCredential {
      */
     constructor(clientIdOrOptions, options) {
         this.isEndpointUnavailable = null;
+        let _options;
         if (typeof clientIdOrOptions === "string") {
             // clientId, options constructor
             this.clientId = clientIdOrOptions;
-            this.identityClient = new IdentityClient(options);
+            _options = options;
         }
         else {
             // options only constructor
-            this.identityClient = new IdentityClient(clientIdOrOptions);
+            _options = options;
         }
+        this.identityClient = new IdentityClient(_options);
+        this.isAvailableIdentityClient = new IdentityClient(Object.assign(Object.assign({}, _options), { retryOptions: {
+                maxRetries: 0,
+            } }));
     }
     async cachedAvailableMSI(scopes, clientId, getTokenOptions) {
         if (this.cachedMSI) {
@@ -2785,7 +2795,7 @@ class ManagedIdentityCredential {
         }
         const MSIs = [fabricMsi, appServiceMsi2017, cloudShellMsi, arcMsi, tokenExchangeMsi(), imdsMsi];
         for (const msi of MSIs) {
-            if (await msi.isAvailable(scopes, this.identityClient, clientId, getTokenOptions)) {
+            if (await msi.isAvailable(scopes, this.isAvailableIdentityClient, clientId, getTokenOptions)) {
                 this.cachedMSI = msi;
                 return msi;
             }