Add user_route.format endpoint route for primary verb in user-defined routes

mythz · mythz · commit 44b61bb5b642 · 2026-02-20T12:10:23.000+08:00
diff --git a/ServiceStack/src/ServiceStack/AppHostBase.NetCore.cs b/ServiceStack/src/ServiceStack/AppHostBase.NetCore.cs
@@ -348,30 +348,31 @@ Task HandleRequestAsync(Type requestType, HttpContext httpContext)
                         {
                             handler(pathBuilder, operation, routeVerb, route.Path);
                         }
-                    }
-                    
-                    // Add /custom/path.{format} routes for GET requests
-                    if (routeVerbs.Contains(HttpMethods.Get) && !route.Path.Contains('.') && !route.Path.Contains('*'))
-                    {
-                        var routePath = route.Path + ".{format}";
-                        if (existingRoutes.TryGetValue(routePath, out var prevRoute))
-                        {
-                            LogManager.GetLogger(GetType()).WarnFormat("Ignoring registering duplicate route: {0} for {1} and {2}", 
-                                routePath,
-                                route.RequestType.FullName,
-                                prevRoute.RequestType.FullName);
-                        }
-                        else
+                        
+                        // Add /custom/path.{format} routes
+                        if (!route.Path.Contains('.') && !route.Path.Contains('*'))
                         {
-                            routeRule = $"[GET] {routePath}";
-                            existingRoutes[routePath] = route;
-                            var pathBuilder = routeBuilder.MapMethods(routePath, EndpointVerbs[HttpMethods.Get], 
-                                (string format, HttpResponse response, HttpContext httpContext) =>
-                                    HandleRequestAsync(requestType, httpContext));
+                            var routePath = route.Path + ".{format}";
+                            if (existingRoutes.TryGetValue(routePath, out var prevRoute))
+                            {
+                                LogManager.GetLogger(GetType()).WarnFormat("Ignoring registering duplicate route: {0} for {1} and {2}", 
+                                    routePath,
+                                    route.RequestType.FullName,
+                                    prevRoute.RequestType.FullName);
+                            }
+                            else
+                            {
+                                routeRule = $"[{verb}] {routePath}";
+                                existingRoutes[routePath] = route;
+                                var pathBuilderFmt = routeBuilder.MapMethods(routePath, verb, 
+                                    (string format, HttpResponse response, HttpContext httpContext) =>
+                                        HandleRequestAsync(requestType, httpContext));
                         
-                            ConfigureOperationEndpoint(pathBuilder, operation, options)
-                                .WithMetadata<string>(routePath);
+                                ConfigureOperationEndpoint(pathBuilderFmt, operation, options)
+                                    .WithMetadata<string>(routePath);
+                            }
                         }
+
                     }
                 }
                 catch (Exception e)
diff --git a/ServiceStack/tests/OpenApi3Swashbuckle/Program.cs b/ServiceStack/tests/OpenApi3Swashbuckle/Program.cs
@@ -32,6 +32,8 @@
 app.UseHttpsRedirection();
 app.UseAuthorization();
 
-app.UseServiceStack(new AppHost());
+app.UseServiceStack(new AppHost(), options => {
+    options.MapEndpoints();
+});
 
 app.Run();
diff --git a/ServiceStack/tests/OpenApi3Swashbuckle/ServiceInterface/MyServices.cs b/ServiceStack/tests/OpenApi3Swashbuckle/ServiceInterface/MyServices.cs
@@ -19,5 +19,10 @@ public class MyServices : Service
     {
         Result = $"Hello, {request.Name ?? "World"}!"
     };
+
+    public object Any(HelloPost request) => new HelloResponse
+    {
+        Result = $"Hello, {request.Name ?? "World"}!"
+    };
 }
 
diff --git a/ServiceStack/tests/OpenApi3Swashbuckle/ServiceModel/Hello.cs b/ServiceStack/tests/OpenApi3Swashbuckle/ServiceModel/Hello.cs
@@ -1,3 +1,5 @@
+using System.Runtime.Serialization;
+
 using ServiceStack;
 
 namespace MyApp.ServiceModel;
@@ -29,4 +31,12 @@ public class HelloSecure : IGet, IReturn<HelloResponse>
 public class HelloApiKey : IGet, IReturn<HelloResponse>
 {
     public string? Name { get; set; }
-}
+}
+
+[Tag("hello")]
+[Route("/hello_post", "POST")]
+public class HelloPost : IPost, IReturn<HelloResponse>
+{
+    [DataMember]
+    public string? Name { get; set; }
+}
diff --git a/ServiceStack/tests/OpenApi3Swashbuckle/hello_xml.http b/ServiceStack/tests/OpenApi3Swashbuckle/hello_xml.http
@@ -0,0 +1,20 @@
+### POST hello_post.xml
+POST https://localhost:5001/hello_post.xml
+Content-Type: application/xml
+
+<HelloPost xmlns:i="http://www.w3.org/2001/XMLSchema-instance"
+           xmlns="http://schemas.datacontract.org/2004/07/MyApp.ServiceModel">
+    <Name>John Doe</Name>
+</HelloPost>
+###
+
+### POST /api/HelloPost.xml
+POST https://localhost:5001/api/HelloPost.xml
+Content-Type: application/xml
+
+<HelloPost xmlns:i="http://www.w3.org/2001/XMLSchema-instance"
+           xmlns="http://schemas.datacontract.org/2004/07/MyApp.ServiceModel">
+    <Name>John Doe</Name>
+</HelloPost>
+###
+

Original file line number	Diff line number	Diff line change
`@@ -19,5 +19,10 @@ public class MyServices : Service`
`19`	`19`	`{`
`20`	`20`	`Result = $"Hello, {request.Name ?? "World"}!"`
`21`	`21`	`};`
	`22`	`+`
	`23`	`+ public object Any(HelloPost request) => new HelloResponse`
	`24`	`+ {`
	`25`	`+ Result = $"Hello, {request.Name ?? "World"}!"`
	`26`	`+ };`
`22`	`27`	`}`
`23`	`28`