Add API docs to healthchecks and CORS (#26416)

* Add API docs to healthchecks and CORS

* Clean up

* Missing docs

Author: JamesNK

src/HealthChecks/Abstractions/src/HealthCheckContext.cs

@@ -3,6 +3,9 @@
 
 namespace Microsoft.Extensions.Diagnostics.HealthChecks
 {
+    /// <summary>
+    /// Health check context. Provides health check registrations to <see cref="IHealthCheck.CheckHealthAsync(HealthCheckContext, System.Threading.CancellationToken)"/>.
+    /// </summary>
     public sealed class HealthCheckContext
     {
         /// <summary>

src/HealthChecks/Abstractions/src/Microsoft.Extensions.Diagnostics.HealthChecks.Abstractions.csproj

@@ -9,7 +9,7 @@ Microsoft.Extensions.Diagnostics.HealthChecks.IHealthCheck
     <RootNamespace>Microsoft.Extensions.Diagnostics.HealthChecks</RootNamespace>
     <TargetFrameworks>$(DefaultNetFxTargetFramework);netstandard2.0;$(DefaultNetCoreTargetFramework)</TargetFrameworks>
     <TargetFrameworks Condition="'$(DotNetBuildFromSource)' == 'true'">$(DefaultNetCoreTargetFramework)</TargetFrameworks>
-    <NoWarn>$(NoWarn);CS1591</NoWarn>
+    <NoWarn>$(NoWarn.Replace('1591', ''))</NoWarn>
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <PackageTags>diagnostics;healthchecks</PackageTags>
     <IsAspNetCoreApp>true</IsAspNetCoreApp>

src/HealthChecks/HealthChecks/src/HealthCheckPublisherOptions.cs

@@ -13,6 +13,9 @@ public sealed class HealthCheckPublisherOptions
         private TimeSpan _delay;
         private TimeSpan _period;
 
+        /// <summary>
+        /// Creates a new instance of <see cref="HealthCheckPublisherOptions"/>.
+        /// </summary>
         public HealthCheckPublisherOptions()
         {
             _delay = TimeSpan.FromSeconds(5);

src/HealthChecks/HealthChecks/src/Microsoft.Extensions.Diagnostics.HealthChecks.csproj

@@ -8,7 +8,7 @@ Microsoft.Extensions.Diagnostics.HealthChecks.IHealthChecksBuilder
     </Description>
     <TargetFrameworks>$(DefaultNetFxTargetFramework);netstandard2.0;$(DefaultNetCoreTargetFramework)</TargetFrameworks>
     <TargetFrameworks Condition="'$(DotNetBuildFromSource)' == 'true'">$(DefaultNetCoreTargetFramework)</TargetFrameworks>
-    <NoWarn>$(NoWarn);CS1591</NoWarn>
+    <NoWarn>$(NoWarn.Replace('1591', ''))</NoWarn>
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <PackageTags>diagnostics;healthchecks</PackageTags>
     <IsAspNetCoreApp>true</IsAspNetCoreApp>

src/Middleware/CORS/src/CorsPolicyMetadata.cs

@@ -10,6 +10,10 @@ namespace Microsoft.AspNetCore.Cors
     /// </summary>
     public class CorsPolicyMetadata : ICorsPolicyMetadata
     {
+        /// <summary>
+        /// Creates a new instance of <see cref="CorsPolicyMetadata"/> using the specified policy.
+        /// </summary>
+        /// <param name="policy">The policy which needs to be applied.</param>
         public CorsPolicyMetadata(CorsPolicy policy)
         {
             Policy = policy;

src/Middleware/CORS/src/Infrastructure/CorsOptions.cs

@@ -19,6 +19,9 @@ public class CorsOptions
         internal IDictionary<string, (CorsPolicy policy, Task<CorsPolicy> policyTask)> PolicyMap { get; }
             = new Dictionary<string, (CorsPolicy, Task<CorsPolicy>)>(StringComparer.Ordinal);
 
+        /// <summary>
+        /// Gets or sets the default policy name.
+        /// </summary>
         public string DefaultPolicyName
         {
             get => _defaultPolicyName;

src/Middleware/CORS/src/Infrastructure/CorsService.cs

@@ -139,11 +139,23 @@ private static void PopulateResult(HttpContext context, CorsPolicy policy, CorsR
             AddHeaderValues(result.AllowedHeaders, allowedHeaders);
         }
 
+        /// <summary>
+        /// Evaluate a request using the specified policy. The result is set on the specified <see cref="CorsResult"/> instance.
+        /// </summary>
+        /// <param name="context">The current HTTP context.</param>
+        /// <param name="policy">The <see cref="CorsPolicy"/> to evaluate.</param>
+        /// <param name="result">The <see cref="CorsResult"/> to set the result on.</param>
         public virtual void EvaluateRequest(HttpContext context, CorsPolicy policy, CorsResult result)
         {
             PopulateResult(context, policy, result);
         }
 
+        /// <summary>
+        /// Evaluate a preflight request using the specified policy. The result is set on the specified <see cref="CorsResult"/> instance.
+        /// </summary>
+        /// <param name="context">The current HTTP context.</param>
+        /// <param name="policy">The <see cref="CorsPolicy"/> to evaluate.</param>
+        /// <param name="result">The <see cref="CorsResult"/> to set the result on.</param>
         public virtual void EvaluatePreflightRequest(HttpContext context, CorsPolicy policy, CorsResult result)
         {
             PopulateResult(context, policy, result);

src/Middleware/CORS/src/Microsoft.AspNetCore.Cors.csproj

@@ -7,7 +7,7 @@ Microsoft.AspNetCore.Cors.DisableCorsAttribute
 Microsoft.AspNetCore.Cors.EnableCorsAttribute</Description>
     <TargetFramework>$(DefaultNetCoreTargetFramework)</TargetFramework>
     <IsAspNetCoreApp>true</IsAspNetCoreApp>
-    <NoWarn>$(NoWarn);CS1591</NoWarn>
+    <NoWarn>$(NoWarn.Replace('1591', ''))</NoWarn>
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <PackageTags>aspnetcore;cors</PackageTags>
     <IsPackable>false</IsPackable>

src/Middleware/HealthChecks.EntityFrameworkCore/src/DependencyInjection/EntityFrameworkCoreHealthChecksBuilderExtensions.cs

@@ -1,7 +1,6 @@
-// Copyright (c) .NET Foundation. All rights reserved.
+// Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-
 using System;
 using System.Collections.Generic;
 using System.Threading;
@@ -13,6 +12,9 @@
 
 namespace Microsoft.Extensions.DependencyInjection
 {
+    /// <summary>
+    /// <see cref="IHealthChecksBuilder"/> extension methods for Entity Framework Core.
+    /// </summary>
     public static class EntityFrameworkCoreHealthChecksBuilderExtensions
     {
         /// <summary>

src/Middleware/HealthChecks.EntityFrameworkCore/src/Microsoft.Extensions.Diagnostics.HealthChecks.EntityFrameworkCore.csproj

@@ -5,7 +5,7 @@
       Components for performing health checks using EntityFrameworkCore.
     </Description>
     <TargetFramework>netstandard2.1</TargetFramework>
-    <NoWarn>$(NoWarn);CS1591</NoWarn>
+    <NoWarn>$(NoWarn.Replace('1591', ''))</NoWarn>
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <PackageTags>diagnostics;healthchecks;entityframeworkcore</PackageTags>
     <BaseNamespace>Microsoft.Extensions.Diagnostics.HealthChecks</BaseNamespace>