Browse Source

Improve the doc comment for MaxAccepts (#28166)

* Improve the doc comment for MaxAccepts

- Added defaults
- Explain that it's about concurrent

* Update src/Servers/HttpSys/src/HttpSysOptions.cs
David Fowler 5 years ago
parent
commit
2d1e963637
1 changed files with 4 additions and 1 deletions
  1. 4 1
      src/Servers/HttpSys/src/HttpSysOptions.cs

+ 4 - 1
src/Servers/HttpSys/src/HttpSysOptions.cs

@@ -68,8 +68,11 @@ namespace Microsoft.AspNetCore.Server.HttpSys
         public ClientCertificateMethod ClientCertificateMethod { get; set; } = ClientCertificateMethod.AllowCertificate;
 
         /// <summary>
-        /// The maximum number of concurrent accepts.
+        /// The number of concurrent workers draining requests from the Http.sys queue.
         /// </summary>
+        /// <remarks>
+        /// Defaults to 5 times the number of processors.
+        /// </remarks>
         public int MaxAccepts { get; set; } = DefaultMaxAccepts;
 
         /// <summary>