fix: strip trailing /index from urls

roxiness · Aug 19, 2023 · 7dde649 · 7dde649
1 parent a64844f
commit 7dde649
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/lib/runtime/Router/Router.js b/lib/runtime/Router/Router.js
@@ -26,6 +26,7 @@ import {
     createSequenceHooksCollection,
 } from 'hookar'
 import reset from '../plugins/reset/index.js'
+import stripTrailingIndex from '../plugins/stripTrailingIndex'
 import scroller from '../plugins/scroller/plugin.js'
 
 import { next } from '../../common/utils.js'
@@ -36,7 +37,7 @@ import { URIDecodeObject } from '../Route/utils.js'
 const stripNullFields = obj =>
     Object.fromEntries(Object.entries(obj).filter(([_, v]) => v != null))
 
-const defaultPlugins = [reset(), scroller()]
+const defaultPlugins = [reset(), scroller(), stripTrailingIndex()]
 
 /**
  * @implements { Readable<Router> }

diff --git a/lib/runtime/plugins/stripTrailingIndex/index.js b/lib/runtime/plugins/stripTrailingIndex/index.js
@@ -0,0 +1,12 @@
+/**
+ * Strips trailing /index from a url
+ *  @returns {RoutifyRuntimePlugin}
+ **/
+export default () => ({
+    urlRewrite: [
+        {
+            toExternal: url => url.replace(/\/index(?=$|\?|#)/, ''),
+            toInternal: url => url,
+        },
+    ],
+})