fix: setup history listeners once

Fix #1344

diff --git a/src/router.ts b/src/router.ts
index 9a87439b00e500c7d8d29c5c22d5fdfbd3c567c6..dee1f9b9df4af4e993b91a4768d35395ff64a253 100644 (file)
--- a/src/router.ts
+++ b/src/router.ts
@@ -938,9 +938,11 @@ export function createRouter(options: RouterOptions): Router {
     markAsReady()
   }
 
-  let removeHistoryListener: () => void | undefined
+  let removeHistoryListener: undefined | null | (() => void)
   // attach listener to history to trigger navigations
   function setupListeners() {
+    // avoid setting up listeners twice due to an invalid first navigation
+    if (removeHistoryListener) return
     removeHistoryListener = routerHistory.listen((to, _from, info) => {
       // cannot be a redirect route because it was in history
       const toLocation = resolve(to) as RouteLocationNormalized
@@ -1220,6 +1222,7 @@ export function createRouter(options: RouterOptions): Router {
           // invalidate the current navigation
           pendingLocation = START_LOCATION_NORMALIZED
           removeHistoryListener && removeHistoryListener()
+          removeHistoryListener = null
           currentRoute.value = START_LOCATION_NORMALIZED
           started = false
           ready = false