Listener options fixes

Author: ManickaP

xml/System.Net.Quic/QuicListener.xml

@@ -69,9 +69,10 @@ For QUIC prerequisites and supported operating systems, see [Platform dependenci
 
 ## Remarks
 
-<xref:System.Net.Quic.QuicListener> doesn't have a mechanism to report inbound connections that fail the handshake process. Such connections are only logged by the listener and never surfaced on the outside.
-
-This method propagates exceptions from <xref:System.Net.Quic.QuicListenerOptions.ConnectionOptionsCallback>, including validation errors from misconfigured <xref:System.Net.Quic.QuicServerConnectionOptions>, for example, <xref:System.ArgumentException>. It also propagates exceptions from failed connection handshakes, for example, <xref:System.Security.Authentication.AuthenticationException> and <xref:System.Net.Quic.QuicException>.
+This method also propagates exceptions from the process of establishing a connection, including:
+- Errors from <xref:System.Net.Quic.QuicListenerOptions.ConnectionOptionsCallback> as <xref:System.Net.Quic.QuicException> with <xref:System.Net.Quic.QuicError.CallbackError> error type.
+- Validation errors from misconfigured <xref:System.Net.Quic.QuicServerConnectionOptions> returned by <xref:System.Net.Quic.QuicListenerOptions.ConnectionOptionsCallback> as <xref:System.ArgumentException>.
+- Exceptions from failed connection handshakes as <xref:System.Security.Authentication.AuthenticationException> or <xref:System.Net.Quic.QuicException>.
 
           ]]></format>
         </remarks>

xml/System.Net.Quic/QuicListenerOptions.xml

@@ -22,7 +22,7 @@
     </Attribute>
   </Attributes>
   <Docs>
-    <summary>Options to provide to a <see cref="T:System.Net.Quic.QuicListener" />.</summary>
+    <summary>Options to configure a new <see cref="T:System.Net.Quic.QuicListener" />.</summary>
     <remarks>To be added.</remarks>
   </Docs>
   <Members>
@@ -65,7 +65,7 @@
       <Docs>
         <summary>Gets or sets the list of application protocols that the listener will accept. At least one must be specified.</summary>
         <value>To be added.</value>
-        <remarks>This property is mandatory, and not setting it will result in validation errors when starting the listener.</remarks>
+        <remarks>This property is mandatory, not setting or not specifying at least one value will result in validation errors when starting the listener.</remarks>
       </Docs>
     </Member>
     <Member MemberName="ConnectionOptionsCallback">
@@ -92,9 +92,9 @@
         <ReturnType>System.Func&lt;System.Net.Quic.QuicConnection,System.Net.Security.SslClientHelloInfo,System.Threading.CancellationToken,System.Threading.Tasks.ValueTask&lt;System.Net.Quic.QuicServerConnectionOptions&gt;&gt;</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Gets or sets the selection callback to choose inbound connection options dynamically.</summary>
+        <summary>Gets or sets the selection callback to choose inbound connection options.</summary>
         <value>To be added.</value>
-        <remarks>To be added.</remarks>
+        <remarks>This property is mandatory, not setting it will result in validation errors when starting the listener.</remarks>
       </Docs>
     </Member>
     <Member MemberName="ListenBacklog">
@@ -115,7 +115,7 @@
         <ReturnType>System.Int32</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Gets or sets the number of connections to be held without accepting any of them, that is, the maximum size of the pending connection queue.</summary>
+        <summary>Gets or sets the number of connections to be held without accepting any of them. That is, the maximum size of the pending connection queue.</summary>
         <value>To be added.</value>
         <remarks>To be added.</remarks>
       </Docs>
@@ -140,7 +140,7 @@
       <Docs>
         <summary>Gets or sets the endpoint to listen on.</summary>
         <value>To be added.</value>
-        <remarks>This property is mandatory, and not setting it will result in validation errors when starting the listener.</remarks>
+        <remarks>This property is mandatory, not setting will result in validation errors when starting the listener.</remarks>
       </Docs>
     </Member>
   </Members>