fix: loop redirection on differentDomains (#2318)

specs/fixtures/issues/2313/app.vue

@@ -0,0 +1,5 @@
+<template>
+  <div>
+    <NuxtPage />
+  </div>
+</template>

specs/fixtures/issues/2313/nuxt.config.ts

@@ -0,0 +1,20 @@
+export default defineNuxtConfig({
+  modules: ['@nuxtjs/i18n'],
+  i18n: {
+    defaultLocale: 'fr',
+    differentDomains: true,
+    locales: [
+      {
+        code: 'en',
+        domain: 'en.nuxt-app.localhost'
+      },
+      {
+        code: 'fr',
+        domain: 'fr.nuxt-app.localhost'
+      }
+    ],
+    strategy: 'prefix'
+    // detectBrowserLanguage: false
+  },
+  ssr: false
+})

specs/fixtures/issues/2313/package.json

@@ -0,0 +1,14 @@
+{
+  "name": "nuxt3-test-issues-2313",
+  "private": true,
+  "scripts": {
+    "build": "nuxt build",
+    "dev": "nuxt dev",
+    "generate": "nuxt generate",
+    "preview": "nuxt preview"
+  },
+  "devDependencies": {
+    "@nuxtjs/i18n": "latest",
+    "nuxt": "latest"
+  }
+}

specs/fixtures/issues/2313/pages/index.vue

@@ -0,0 +1,9 @@
+<template>
+  <div>
+    Locale: <code id="locale">{{ locale }}</code>
+  </div>
+</template>
+
+<script setup lang="ts">
+const { locale } = useI18n()
+</script>

specs/issues/2313.spec.ts

@@ -0,0 +1,28 @@
+import { test, expect, describe } from 'vitest'
+import { fileURLToPath } from 'node:url'
+import { setup, url, createPage } from '../utils'
+import { getText } from '../helper'
+
+describe('#2313', async () => {
+  await setup({
+    rootDir: fileURLToPath(new URL(`../fixtures/issues/2313`, import.meta.url))
+  })
+
+  async function assertDifferenctDomain(locale: string, browserLocale = 'en') {
+    const home = url('/')
+    const page = await createPage(undefined, {
+      extraHTTPHeaders: {
+        'X-Forwarded-Host': `${locale}.nuxt-app.localhost`
+      },
+      locale: browserLocale
+    })
+    await page.goto(home)
+    expect(await getText(page, '#locale')).toEqual(locale)
+    await page.close()
+  }
+
+  test('detection locale from domain', async () => {
+    await assertDifferenctDomain('en')
+    await assertDifferenctDomain('fr', 'fr')
+  })
+})

src/runtime/internal.ts

@@ -512,7 +512,7 @@ export function getDomainFromLocale(localeCode: Locale, locales: LocaleObject[],
       } = useRequestEvent(nuxt)
       protocol = req && isHTTPS(req) ? 'https' : 'http'
     } else {
-      protocol = window.location.protocol.split(':')[0]
+      protocol = new URL(window.location.origin).protocol
     }
     return protocol + '://' + lang.domain
   }

src/runtime/utils.ts

@@ -422,7 +422,7 @@ export async function navigate<Context extends NuxtApp = NuxtApp>(
     const state = useRedirectState()
     __DEBUG__ && console.log('redirect state ->', state.value, 'redirectPath -> ', redirectPath)
     if (process.client) {
-      if (state.value !== redirectPath) {
+      if (state.value && state.value !== redirectPath) {
         state.value = '' // reset redirect path
         window.location.assign(redirectPath)
       }