Fix Az.Accounts XML Comments

src/Accounts/Authentication.ResourceManager/AzureRmProfile.cs

@@ -237,6 +237,7 @@ public AzureRmProfile()
         /// Initializes a new instance of AzureRMProfile and loads its content from specified path.
         /// </summary>
         /// <param name="path">The location of profile file on disk.</param>
+        /// <param name="shouldRefreshContextsFromCache"></param>
         public AzureRmProfile(string path, bool shouldRefreshContextsFromCache = true) : this()
         {
             this.ShouldRefreshContextsFromCache = shouldRefreshContextsFromCache;
@@ -267,6 +268,7 @@ public void Save()
         /// Writes profile to a specified path.
         /// </summary>
         /// <param name="path">File path on disk to save profile to</param>
+        /// <param name="serializeCache"></param>
         public void Save(string path, bool serializeCache = true)
         {
             if (string.IsNullOrEmpty(path))
@@ -284,6 +286,7 @@ public void Save(string path, bool serializeCache = true)
         /// Writes the profile using the specified file provider
         /// </summary>
         /// <param name="provider">The file provider used to save the profile</param>
+        /// <param name="serializeCache"></param>
         public void Save(IFileProvider provider, bool serializeCache = true)
         {
             foreach (string env in AzureEnvironment.PublicEnvironments.Keys)

src/Accounts/Authentication.ResourceManager/Models/PSAzureContext.cs

@@ -164,8 +164,7 @@ public PSAzureContext(PSObject other)
         public IAzureTenant Tenant { get; set; }
 
         /// <summary>
-        /// Moved to <see cref="IClientApplicationBase.ClientTokenCache"> due to MSAL.
-        /// See <see cref="AuthenticationClientFactory"> for how to create client applications.
+        /// Moved to <see cref="TokenCache"/> due to MSAL.
         /// </summary>
         public IAzureTokenCache TokenCache { get; set; } = null;
 

src/Accounts/Authentication.ResourceManager/Models/PSAzureSubscriptionPolicy.cs

@@ -16,7 +16,7 @@ public PSAzureSubscriptionPolicy()
         /// <summary>
         /// object constructor
         /// </summary>
-        /// <param name="azureSubscriptionPolicies">Json string to convert</param>
+        /// <param name="subscriptionPolicies">Json string to convert</param>
         public PSAzureSubscriptionPolicy(SubscriptionPolicies subscriptionPolicies)
         {
             if (subscriptionPolicies != null)

src/Accounts/Authentication/AdalLogger.cs

@@ -131,6 +131,7 @@ internal static void Disable(AdalLogger logger)
             /// </summary>
             /// <param name="level">The log level</param>
             /// <param name="message">The log message</param>
+            /// <param name="containsPII"></param>
             public void Log(LogLevel level, string message, bool containsPII)
             {
                 foreach (var logger in _loggers)

src/Accounts/Authentication/Authentication/IClaimsChallengeProcessor.cs

@@ -29,7 +29,7 @@ public interface IClaimsChallengeProcessor
         /// <param name="request">The origin request that responds with a claim challenge</param>
         /// <param name="claimsChallenge">Claims challenge string</param>
         /// <param name="cancellationToken">Cancelation token</param>
-        /// <returns>Successful or not
+        /// <returns>Successful or not</returns>
         ValueTask<bool> OnClaimsChallenageAsync(HttpRequestMessage request, string claimsChallenge, CancellationToken cancellationToken);
     }
 }

src/Accounts/Authentication/ClaimsChallengeHandler.cs

@@ -59,7 +59,9 @@ public virtual object Clone()
         /// Executed in the event a 401 response with a WWW-Authenticate authentication challenge header is received after the initial request.
         /// </summary>
         /// <remarks>This implementation handles common authentication challenges such as claims challenges. Service client libraries may derive from this and extend to handle service specific authentication challenges.</remarks>
-        /// <param name="message">The HttpMessage to be authenticated.</param>
+        /// <param name="requestMessage">The HttpMessage to be authenticated.</param>
+        /// <param name="cancellationToken"></param>
+        /// <param name="responseMessage"></param>
         /// <returns>A boolean indicated whether the request should be retried</returns>
         protected virtual async Task<bool> OnChallengeAsync(HttpRequestMessage requestMessage, HttpResponseMessage responseMessage, CancellationToken cancellationToken)
         {

src/Accounts/Authentication/Config/Internal/ConfigurationBuilder.cs

@@ -39,6 +39,7 @@ internal class ConfigurationBuilder : IConfigurationBuilder
         /// <summary>
         /// Adds a new configuration source.
         /// </summary>
+        /// <param name="id"></param>
         /// <param name="source">The configuration source to add.</param>
         /// <returns>The same <see cref="IConfigurationBuilder"/>.</returns>
         public IConfigurationBuilder Add(string id, IConfigurationSource source)

src/Accounts/Authentication/Config/Internal/ConfigurationExtensions.cs

@@ -28,6 +28,7 @@ internal static class ConfigurationExtensions
         /// Adds a new configuration source.
         /// </summary>
         /// <param name="builder">The <see cref="IConfigurationBuilder"/> to add to.</param>
+        /// <param name="id"></param>
         /// <param name="configureSource">Configures the source secrets.</param>
         /// <returns>The <see cref="IConfigurationBuilder"/>.</returns>
         public static IConfigurationBuilder Add<TSource>(this IConfigurationBuilder builder, string id, Action<TSource> configureSource) where TSource : IConfigurationSource, new()

src/Accounts/Authentication/Config/Internal/Interfaces/IConfigurationBuilder.cs

@@ -35,6 +35,7 @@ internal interface IConfigurationBuilder
         /// <summary>
         /// Adds a new configuration source.
         /// </summary>
+        /// <param name="id"></param>
         /// <param name="source">The configuration source to add.</param>
         /// <returns>The same <see cref="IConfigurationBuilder"/>.</returns>
         IConfigurationBuilder Add(string id, IConfigurationSource source);

src/Accounts/Authentication/Config/Internal/Interfaces/IConfigurationSource.cs

@@ -23,6 +23,7 @@ internal interface IConfigurationSource
         /// Builds the <see cref="IConfigurationProvider"/> for this source.
         /// </summary>
         /// <param name="builder">The <see cref="IConfigurationBuilder"/>.</param>
+        /// <param name="id"></param>
         /// <returns>An <see cref="IConfigurationProvider"/></returns>
         IConfigurationProvider Build(IConfigurationBuilder builder, string id);
     }

src/Accounts/Authentication/Config/Internal/Providers/JsonConfigurationExtensions.cs

@@ -27,6 +27,7 @@ internal static class JsonConfigurationExtensions
         /// Adds a JSON configuration source to <paramref name="builder"/>.
         /// </summary>
         /// <param name="builder">The <see cref="IConfigurationBuilder"/> to add to.</param>
+        /// <param name="id"></param>
         /// <param name="stream">The <see cref="Stream"/> to read the json configuration data from.</param>
         /// <returns>The <see cref="IConfigurationBuilder"/>.</returns>
         public static IConfigurationBuilder AddJsonStream(this IConfigurationBuilder builder, string id, Stream stream)

src/Accounts/Authentication/Config/Internal/Providers/JsonStreamConfigurationSource.cs

@@ -25,6 +25,7 @@ internal class JsonStreamConfigurationSource : StreamConfigurationSource
         /// Builds the <see cref="JsonStreamConfigurationProvider"/> for this source.
         /// </summary>
         /// <param name="builder">The <see cref="IConfigurationBuilder"/>.</param>
+        /// <param name="id"></param>
         /// <returns>An <see cref="JsonStreamConfigurationProvider"/></returns>
         public override IConfigurationProvider Build(IConfigurationBuilder builder, string id)
             => new JsonStreamConfigurationProvider(this, id);

src/Accounts/Authentication/Config/Internal/Providers/StreamConfigurationProvider.cs

@@ -33,6 +33,7 @@ internal abstract class StreamConfigurationProvider : ConfigurationProvider
         /// Constructor.
         /// </summary>
         /// <param name="source">The source.</param>
+        /// <param name="id"></param>
         public StreamConfigurationProvider(StreamConfigurationSource source, string id) : base(id)
         {
             Source = source ?? throw new ArgumentNullException(nameof(source));

src/Accounts/Authentication/Config/Internal/Providers/StreamConfigurationSource.cs

@@ -31,6 +31,7 @@ internal abstract class StreamConfigurationSource : IConfigurationSource
         /// Builds the <see cref="StreamConfigurationProvider"/> for this source.
         /// </summary>
         /// <param name="builder">The <see cref="IConfigurationBuilder"/>.</param>
+        /// <param name="id"></param>
         /// <returns>An <see cref="IConfigurationProvider"/></returns>
         public abstract IConfigurationProvider Build(IConfigurationBuilder builder, string id);
     }

src/Accounts/Authentication/Config/Internal/Providers/UnsettableMemoryConfigurationProvider.cs

@@ -30,6 +30,7 @@ internal class UnsettableMemoryConfigurationProvider : ConfigurationProvider, IE
         /// Initialize a new instance from the source.
         /// </summary>
         /// <param name="source">The source settings.</param>
+        /// <param name="id"></param>
         public UnsettableMemoryConfigurationProvider(UnsettableMemoryConfigurationSource source, string id): base(id)
         {
             if (source == null)

src/Accounts/Authenticators/Identity/ITokenCacheOptions.cs

@@ -27,7 +27,7 @@ public class TokenCachePersistenceOptions
 
         /// <summary>
         /// If set to true the token cache may be persisted as an unencrypted file if no OS level user encryption is available. When set to false the token cache
-        /// will throw a <see cref="CredentialUnavailableException"/> in the event no OS level user encryption is available.
+        /// will throw a <see cref="UnsafeAllowUnencryptedStorage"/> 
         /// </summary>
         public bool UnsafeAllowUnencryptedStorage { get; set; }
     }