Make Federation package generate documentation file. Fix some doc for…

…matting bugs

src/System.ServiceModel.Federation/src/System.ServiceModel.Federation.csproj

@@ -8,6 +8,7 @@
     <IsShipping>$(Ship_WcfPackages)</IsShipping>
     <TargetFramework>net6.0</TargetFramework>
     <PackageDescription>Provides the types that allow security communications with SOAP messages using WS-Federation.</PackageDescription>
+    <GenerateDocumentationFile>true</GenerateDocumentationFile>
   </PropertyGroup>
 
   <ItemGroup>

src/System.ServiceModel.Federation/src/System/ServiceModel/Federation/IWSTrustChannelContract.cs

@@ -19,7 +19,6 @@ public interface IWSTrustChannelContract
         /// </summary>
         /// <param name="request">The <see cref="WsTrustRequest" /> to send to the STS.</param>
         /// <returns>A <see cref="SecurityToken" /> issued by the STS.</returns>
-        /// <summary>
         [OperationContract(Name = "Issue", Action = "*", ReplyAction = "*")]
         Task<SecurityToken> IssueAsync(WsTrustRequest request);
     }

src/System.ServiceModel.Federation/src/System/ServiceModel/Federation/WSTrustChannel.cs

@@ -81,7 +81,7 @@ protected virtual Message CreateRequest(WsTrustRequest trustRequest)
         /// <summary>
         /// Gets the WS-Addressing SOAP action that corresponds to the <see cref="WsTrustRequest"/>.RequestType and <see cref="WsTrustRequest"/>.WsTrustVersion.
         /// </summary>
-        /// <param name="trustRequest">The <see cref="WsTrustRequest"/> to generate the WS-Addressing action.
+        /// <param name="trustRequest">The <see cref="WsTrustRequest"/> to generate the WS-Addressing action.</param>
         /// <returns>The WS-Addressing action to use.</returns>
         public static string GetRequestAction(WsTrustRequest trustRequest)
         {
@@ -112,7 +112,7 @@ public static string GetRequestAction(WsTrustRequest trustRequest)
         /// <summary>
         /// Gets the <see cref="WsSerializationContext"/> to use when serializing the <see cref="WsTrustRequest"/>.
         /// </summary>
-        /// <param name="trustRequest">The <see cref="WsTrustRequest"/> that will be serialized.
+        /// <param name="trustRequest">The <see cref="WsTrustRequest"/> that will be serialized.</param>
         /// <returns>The <see cref="WsSerializationContext"/> for the <see cref="WsTrustRequest"/>.</returns>
         private WsSerializationContext GetSerializationContext(WsTrustRequest trustRequest)
         {

src/System.ServiceModel.Federation/src/System/ServiceModel/Federation/WSTrustUtilities.cs

@@ -20,6 +20,7 @@ namespace System.ServiceModel.Federation
     /// </summary>
     internal static class WSTrustUtilities
     {
+        /// <summary>
         /// Get a proof token from a WsTrust request/response pair based on section 4.4.3 of the WS-Trust 1.3 spec.
         /// How the proof token is retrieved depends on whether the requester or issuer provide key material:
         /// Requester   |   Issuer                  | Results
@@ -32,6 +33,8 @@ internal static class WSTrustUtilities
         /// </summary>
         /// <param name="request">The WS-Trust request (RST).</param>
         /// <param name="response">The WS-Trust response (RSTR).</param>
+        /// <param name="serializationContext">The serialization context.</param>
+        /// <param name="algorithmSuite">The algorithm suite.</param>
         /// <returns>The proof token or null if there is no proof token.</returns>
         internal static BinarySecretSecurityToken GetProofToken(WsTrustRequest request, Microsoft.IdentityModel.Protocols.WsTrust.RequestSecurityTokenResponse response, WsSerializationContext serializationContext, SecurityAlgorithmSuite algorithmSuite)
         {

src/System.ServiceModel.Federation/src/System/ServiceModel/Federation/WsFederationHttpBinding.cs

@@ -45,8 +45,8 @@ public WSTrustTokenParameters WSTrustTokenParameters
         /// </summary>
         /// <returns>a <see cref="SecurityBindingElement"/> for the RelyingParty channel.</returns>
         /// <remarks>Creates a new <see cref="TransportBindingElement"/> and adds <see cref="WSTrustTokenParameters"/> to the appropriate EndpointSupportingTokenParameters (Endorsing or Signed).
-        /// <para>Sets: <see cref="MessageSecurityVersion"/> == <see cref="WSTrustTokenParameters.MessageSecurityVersion"/>.</para></para>
-        /// <para>Sets: <see cref="IncludeTimestamp"/> == true.</para></remarks>
+        /// <para>Sets: <see cref="SecurityBindingElement.MessageSecurityVersion"/> == <see cref="WSTrustTokenParameters.MessageSecurityVersion"/>.</para>
+        /// <para>Sets: <see cref="SecurityBindingElement.IncludeTimestamp"/> == true.</para></remarks>
         protected override SecurityBindingElement CreateMessageSecurity()
         {
             WSTrustTokenParameters.RequireDerivedKeys = false;

src/System.ServiceModel.Federation/src/System/ServiceModel/Federation/WsTrustChannelClientCredentials.cs

@@ -36,7 +36,7 @@ protected WSTrustChannelClientCredentials(WSTrustChannelClientCredentials other)
         /// Crates an instance of <see cref="WSTrustChannelClientCredentials"/> with specifying a <see cref="ClientCredentials"/>
         /// </summary>
         /// <param name="clientCredentials">The <see cref="SecurityTokenManager"/> from this parameter will be used to
-        /// create the <see cref="SecurityTokenProvider"/> in the case the <see cref="SecurityTokenParameters"/> in the channel are not a <see cref="WSTrustTokenParameters"/></para></remarks>
+        /// create the <see cref="SecurityTokenProvider"/> in the case the <see cref="System.ServiceModel.Security.Tokens.SecurityTokenParameters"/> in the channel are not a <see cref="WSTrustTokenParameters"/></param>
         public WSTrustChannelClientCredentials(ClientCredentials clientCredentials)
             : base(clientCredentials)
         {
@@ -62,7 +62,7 @@ protected override ClientCredentials CloneCore()
         /// </summary>
         /// <remarks>The <see cref="SecurityTokenManager"/> is responsible to return the <see cref="SecurityTokenProvider"/> to obtain the issued token.
         /// <para>If <see cref="ClientCredentials"/> was passed to the constructor, then that <see cref="SecurityTokenManager"/> will be used to
-        /// create the <see cref="SecurityTokenProvider"/> in the case the <see cref="SecurityTokenParameters"/> in the channel are not a <see cref="WSTrustTokenParameters"/></para></remarks>
+        /// create the <see cref="SecurityTokenProvider"/> in the case the <see cref="System.ServiceModel.Security.Tokens.SecurityTokenParameters"/> in the channel are not a <see cref="WSTrustTokenParameters"/></para></remarks>
         /// <returns>An instance of <see cref="WSTrustChannelSecurityTokenManager" />.</returns>
         public override SecurityTokenManager CreateSecurityTokenManager()
         {

src/System.ServiceModel.Federation/src/System/ServiceModel/Federation/WsTrustTokenParameters.cs

@@ -31,10 +31,12 @@ public class WSTrustTokenParameters : IssuedSecurityTokenParameters
         private int _issuedTokenRenewalThresholdPercentage = DefaultIssuedTokenRenewalThresholdPercentage;
 
         /// <summary>
-        /// Instantiates a <see cref="wsTrustTokenParameters"/> that describe the parameters for a WSTrust request.
+        /// Instantiates a <see cref="WSTrustTokenParameters"/> that describe the parameters for a WSTrust request.
         /// </summary>
-        /// <remarks><see cref="KeyType"/> == <see cref="SecurityKeyType.SymmetricKey"/>.
-        /// <para><see cref="MessageSecurityVersion"/> == <see cref="MessageSecurityVersion.WSSecurity11WSTrust13WSSecureConversation13WSSecurityPolicy12BasicSecurityProfile10"/></para></remarks>
+        /// <remarks>
+        /// <para>Sets: <see cref="IssuedSecurityTokenParameters.KeyType"/> == <see cref="SecurityKeyType.SymmetricKey"/>.</para>
+        /// <para>Sets: <see cref="IssuedSecurityTokenParameters.DefaultMessageSecurityVersion"/> == <see cref="MessageSecurityVersion.WSSecurity11WSTrust13WSSecureConversation13WSSecurityPolicy12BasicSecurityProfile10"/></para>
+        /// <para>Sets: <see cref="MessageSecurityVersion"/> == <see cref="MessageSecurityVersion.WSSecurity11WSTrust13WSSecureConversation13WSSecurityPolicy12BasicSecurityProfile10"/></para></remarks>
         public WSTrustTokenParameters()
         {
             KeyType = DefaultSecurityKeyType;