Merge of #642. Thanks @andregouveia8

pnp · Jun 8, 2022 · dcaa3b6 · dcaa3b6
1 parent f758cf3
commit dcaa3b6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/lib/CHANGELOG.md b/src/lib/CHANGELOG.md
@@ -24,6 +24,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
 - Extend timeouts and add retries to teams provisioning #669 [patrikhellgren - Patrik Hellgren]
 - Implement show / hide of site title in header using SetChromeOptions #670 [eduardpaul - Eduard Paul]
 - Improve folder creation in EnsureFolderPath method #673 [gautamdsheth - Gautam Sheth]
+- Add UserAgent to CSOM based access token retrieval in GetAccessToken #642 [andregouveia8 - André Gouveia]
 
 ## [1.9.0]
 

diff --git a/src/lib/PnP.Framework/Extensions/ClientContextExtensions.cs b/src/lib/PnP.Framework/Extensions/ClientContextExtensions.cs
@@ -666,10 +666,7 @@ public static string GetAccessToken(this ClientRuntimeContext clientContext)
                             accessToken = authorization.Replace("Bearer ", string.Empty);
                         }
 
-                        if (!string.IsNullOrWhiteSpace(userAgentFromConfig))
-                        {
-                            e.WebRequestExecutor.WebRequest.Headers.Add("User-Agent", userAgentFromConfig);
-                        }
+                        e.WebRequestExecutor.WebRequest.UserAgent = string.IsNullOrEmpty(userAgentFromConfig) ? $"{PnPCoreUtilities.PnPCoreUserAgent}" : userAgentFromConfig;
                     };
                     // Issue a dummy request to get it from the Authorization header
                     clientContext.ExecutingWebRequest += handler;