Change additional auth context type and log at appropriate step

microsoft · Oct 30, 2024 · 4c1f70c · 4c1f70c
1 parent 6db569f
commit 4c1f70c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/http/httpClient/Middleware/AuthorizationHandler.cs b/src/http/httpClient/Middleware/AuthorizationHandler.cs
@@ -85,7 +85,7 @@ protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage
         }
 
         private async Task AuthenticateRequestAsync(HttpRequestMessage request,
-            Dictionary<string, object>? additionalAuthenticationContext,
+            Dictionary<string, object> additionalAuthenticationContext,
             CancellationToken cancellationToken,
             Activity? activityForAttributes)
         {
@@ -98,8 +98,8 @@ private async Task AuthenticateRequestAsync(HttpRequestMessage request,
             var accessToken = await accessTokenProvider.GetAuthorizationTokenAsync(
                 request.RequestUri,
                 additionalAuthenticationContext, cancellationToken).ConfigureAwait(false);
-            if(string.IsNullOrEmpty(accessToken)) return;
             activityForAttributes?.SetTag("com.microsoft.kiota.handler.authorization.token_obtained", true);
+            if(string.IsNullOrEmpty(accessToken)) return;
             request.Headers.TryAddWithoutValidation(AuthorizationHeader, $"Bearer {accessToken}");
         }
     }