fix: discard params in path redirect

Fix #1401

diff --git a/__tests__/warnings.spec.ts b/__tests__/warnings.spec.ts
index e749472b741ea52171fa675600980caad4382dbb..b6ac76087e184ea3e50c687172f38375b166ecb5 100644 (file)
--- a/__tests__/warnings.spec.ts
+++ b/__tests__/warnings.spec.ts
@@ -32,7 +32,7 @@ describe('warnings', () => {
       history,
       routes: [{ path: '/:p', name: 'p', component }],
     })
-    router.resolve({ path: '/p', params: { p: 'p' } })
+    router.push({ path: '/p', params: { p: 'p' } })
     expect('Path "/p" was passed with params').toHaveBeenWarned()
   })
 
@@ -42,10 +42,26 @@ describe('warnings', () => {
       history,
       routes: [{ path: '/:p', name: 'p', component }],
     })
-    router.resolve({ path: '/p', name: 'p', params: { p: 'p' } })
+    router.push({ path: '/p', name: 'p', params: { p: 'p' } })
     expect('Path "/" was passed with params').not.toHaveBeenWarned()
   })
 
+  it('does not warn when redirecting from params', async () => {
+    const history = createMemoryHistory()
+    const router = createRouter({
+      history,
+      routes: [
+        {
+          path: '/p/:p',
+          redirect: to => ({ path: '/s', query: { p: to.params.p } }),
+        },
+        { path: '/s', component },
+      ],
+    })
+    router.push({ path: '/p/abc' })
+    expect('was passed with params').not.toHaveBeenWarned()
+  })
+
   it('warns if an alias is missing params', async () => {
     createRouter({
       history: createMemoryHistory(),

diff --git a/src/router.ts b/src/router.ts
index 7f0f9d1f958eba0a4ac3668ef69a28247d2d1240..abd238b7a25067ddd398454eb73dfaaf94a49287 100644 (file)
--- a/src/router.ts
+++ b/src/router.ts
@@ -650,7 +650,8 @@ export function createRouter<Options extends RouterOptions>(
         {
           query: to.query,
           hash: to.hash,
-          params: to.params,
+          // avoid transferring params if the redirect has a path
+          params: 'path' in newTargetLocation ? {} : to.params,
         },
         newTargetLocation
       )