fix(nuxt): encode location header in navigateTo (#26712)

nuxt · Apr 10, 2024 · 30e27f4 · 30e27f4
1 parent a7425d3
commit 30e27f4
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 1 deletion.
diff --git a/packages/nuxt/src/app/composables/router.ts b/packages/nuxt/src/app/composables/router.ts
@@ -169,7 +169,7 @@ export const navigateTo = (to: RouteLocationRaw | undefined | null, options?: Na
         nuxtApp.ssrContext!._renderResponse = {
           statusCode: sanitizeStatusCode(options?.redirectCode || 302, 302),
           body: `<!DOCTYPE html><html><head><meta http-equiv="refresh" content="0; url=${encodedLoc}"></head></html>`,
-          headers: { location },
+          headers: { location: encodeURI(location) },
         }
         return response
       }

diff --git a/test/basic.test.ts b/test/basic.test.ts
@@ -900,6 +900,12 @@ describe('navigate', () => {
 
     expect(status).toEqual(404)
   })
+
+  it('expect to redirect with encoding', async () => {
+    const { status } = await fetch('/redirect-with-encode', { redirect: 'manual' })
+
+    expect(status).toEqual(302)
+  })
 })
 
 describe('preserves current instance', () => {

diff --git a/test/fixtures/basic/pages/redirect-with-encode.vue b/test/fixtures/basic/pages/redirect-with-encode.vue
@@ -0,0 +1,9 @@
+<template>
+  <div>
+    oh no !
+  </div>
+</template>
+
+<script setup lang="ts">
+await navigateTo('/cœur')
+</script>