Add an event handler responsible for mapping non-standard revocation parameters to their standard equivalent

Author: kevinchalet

src/OpenIddict.Client.WebIntegration/OpenIddictClientWebIntegrationHandlers.Revocation.cs

@@ -21,6 +21,7 @@ public static class Revocation
             /*
              * Revocation request preparation:
              */
+            MapNonStandardRequestParameters.Descriptor,
             OverrideHttpMethod.Descriptor,
             AttachBearerAccessToken.Descriptor,
 
@@ -30,6 +31,42 @@ public static class Revocation
             NormalizeContentType.Descriptor
         ]);
 
+        /// <summary>
+        /// Contains the logic responsible for mapping non-standard request parameters
+        /// to their standard equivalent for the providers that require it.
+        /// </summary>
+        public sealed class MapNonStandardRequestParameters : IOpenIddictClientHandler<PrepareRevocationRequestContext>
+        {
+            /// <summary>
+            /// Gets the default descriptor definition assigned to this handler.
+            /// </summary>
+            public static OpenIddictClientHandlerDescriptor Descriptor { get; }
+                = OpenIddictClientHandlerDescriptor.CreateBuilder<PrepareRevocationRequestContext>()
+                    .UseSingletonHandler<MapNonStandardRequestParameters>()
+                    .SetOrder(int.MinValue + 100_000)
+                    .SetType(OpenIddictClientHandlerType.BuiltIn)
+                    .Build();
+
+            /// <inheritdoc/>
+            public ValueTask HandleAsync(PrepareRevocationRequestContext context)
+            {
+                if (context is null)
+                {
+                    throw new ArgumentNullException(nameof(context));
+                }
+
+                // Yandex doesn't support the standard "token" parameter and requires
+                // using the non-standard "access_token" parameter instead.
+                if (context.Registration.ProviderType is ProviderTypes.VkId)
+                {
+                    context.Request.AccessToken = context.Token;
+                    context.Request.Token = null;
+                }
+
+                return default;
+            }
+        }
+
         /// <summary>
         /// Contains the logic responsible for overriding the HTTP method for the providers that require it.
         /// </summary>
@@ -61,7 +98,6 @@ public ValueTask HandleAsync(PrepareRevocationRequestContext context)
 
                 request.Method = context.Registration.ProviderType switch
                 {
-
                     ProviderTypes.Zendesk => HttpMethod.Delete,
 
                     _ => request.Method