fix(swagger): fix redirectMiddleware to be a middleware instead of a …

….get()
tsedio · Feb 29, 2024 · a1f98e8 · a1f98e8
1 parent 37cdc0c
commit a1f98e8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/specs/swagger/src/SwaggerModule.ts b/packages/specs/swagger/src/SwaggerModule.ts
@@ -68,7 +68,7 @@ export class SwaggerModule implements OnRoutesInit, OnReady {
     this.settings.forEach((conf: SwaggerSettings) => {
       const {path = "/"} = conf;
 
-      this.app.get(path, useContextHandler(redirectMiddleware(path)));
+      this.app.use(path, useContextHandler(redirectMiddleware(path)));
       this.app.use(path, this.createRouter(conf, urls));
     });
 

diff --git a/packages/specs/swagger/src/middlewares/redirectMiddleware.ts b/packages/specs/swagger/src/middlewares/redirectMiddleware.ts
@@ -6,7 +6,7 @@ import {PlatformContext} from "@tsed/common";
  */
 export function redirectMiddleware(path: string) {
   return (ctx: PlatformContext) => {
-    if (ctx.request.url === path && !ctx.request.url.match(/\/$/)) {
+    if (ctx.request.method === "GET" && ctx.request.url === path && !ctx.request.url.match(/\/$/)) {
       ctx.response.redirect(302, `${path}/`);
     }
   };