fix: trailing slash urls can discover route rules

nuxt-modules · Jan 5, 2023 · 55b440a · 55b440a
1 parent 904b1d8
commit 55b440a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/.playground/nuxt.config.ts b/.playground/nuxt.config.ts
@@ -22,6 +22,6 @@ export default defineNuxtConfig({
   },
   routeRules: {
     '/secret': { index: false },
-    '/about': { sitemap: { changefreq: 'daily', priority: 0.3 } }
+    '/about': { sitemap: { changefreq: 'daily', priority: 0.3 } },
   }
 })
diff --git a/src/module.ts b/src/module.ts
@@ -113,14 +113,15 @@ declare module 'nitropack' {
           // fix order
           .sort()
           // check route rules
-          .map((path) => {
-            const routeRules = defu({}, ..._routeRulesMatcher.matchAll(path).reverse())
+          .map((route) => {
+            // route matcher assumes all routes have no trailing slash
+            const routeRules = defu({}, ..._routeRulesMatcher.matchAll(withoutTrailingSlash(route)).reverse())
             // @ts-expect-error untyped
             if (routeRules.index === false)
               return false
 
             // @ts-expect-error untyped
-            return { url: fixSlashes(path), ...(routeRules.sitemap || {}) }
+            return { url: route, ...(routeRules.sitemap || {}) }
           })
           .filter(Boolean)