fix: use a client-side navigation when redirecting to a rewriten URL

packages/next/next-server/lib/router/router.ts

@@ -1112,7 +1112,12 @@ export default class Router implements BaseRouter {
               pages
             )
 
-            if (pages.includes(parsedHref.pathname)) {
+            const isDestinationAKnownPage = pages.includes(parsedHref.pathname)
+            const isDestinationARewrittenUrl = rewrites.afterFiles.some(
+              (x: any) => x.source === parsedHref.pathname
+            )
+
+            if (isDestinationAKnownPage || isDestinationARewrittenUrl) {
               const { url: newUrl, as: newAs } = prepareUrlAs(
                 this,
                 destination,

test/integration/redirect-getserversideprops-rewrites/next.config.js

@@ -0,0 +1,10 @@
+module.exports = {
+  async rewrites() {
+    return [
+      {
+        source: '/',
+        destination: '/some-page',
+      },
+    ]
+  },
+}

test/integration/redirect-getserversideprops-rewrites/pages/some-other-page.js

@@ -0,0 +1,16 @@
+export default function SomeOtherPage() {
+  return (
+    <main>
+      <h1>Hello world</h1>
+    </main>
+  )
+}
+
+export const getServerSideProps = ({ query }) => {
+  return {
+    redirect: {
+      destination: `${query.redirect}?message=${query.message}`,
+      permanent: false,
+    },
+  }
+}

test/integration/redirect-getserversideprops-rewrites/pages/some-page.js

@@ -0,0 +1,21 @@
+import Link from 'next/link'
+
+export default function SomePage({ message }) {
+  return (
+    <main>
+      <h1>Hello {message}</h1>
+      <Link href="/some-other-page?redirect=/&message=refreshed">
+        <a id="link-with-rewritten-url" className={message}>
+          Link with rewritten target url
+        </a>
+      </Link>
+      <Link href="/some-other-page?redirect=/some-page">
+        <a>Link with client side navigation</a>
+      </Link>
+    </main>
+  )
+}
+
+export const getServerSideProps = ({ query }) => ({
+  props: { message: query.message || 'World ' },
+})

test/integration/redirect-getserversideprops-rewrites/test/index.test.js

@@ -0,0 +1,57 @@
+/* eslint-env jest */
+
+import { join } from 'path'
+import {
+  renderViaHTTP,
+  findPort,
+  launchApp,
+  killApp,
+  waitFor,
+} from 'next-test-utils'
+import webdriver from 'next-webdriver'
+
+// test suites
+
+const context = {}
+jest.setTimeout(1000 * 60 * 5)
+
+describe('getServerSideProps redirects', () => {
+  beforeAll(async () => {
+    context.appPort = await findPort()
+    context.server = await launchApp(join(__dirname, '../'), context.appPort, {
+      env: { __NEXT_TEST_WITH_DEVTOOL: 1 },
+    })
+
+    // pre-build all pages at the start
+    await Promise.all([
+      renderViaHTTP(context.appPort, '/'),
+      renderViaHTTP(context.appPort, '/some-page'),
+    ])
+  })
+  afterAll(() => killApp(context.server))
+
+  it('should use a client-side navigation for a rewritten URL', async () => {
+    const browser = await webdriver(context.appPort, '/')
+
+    await browser.executeScript(function () {
+      // During a browser navigation global variables are reset,
+      // So by chaking that the __SAME_PAGE variable is still defined
+      // then the client-side navigation has happened
+      window.__SAME_PAGE = true
+    })
+
+    await browser.elementByCss('#link-with-rewritten-url').click()
+
+    // Wait for a potential page reload
+    await waitFor(1000)
+
+    // Wait until the new props are rendered
+    await browser.elementByCss('.refreshed')
+
+    const isSamePage = await browser.executeScript(function () {
+      return window.__SAME_PAGE || false
+    })
+
+    expect(isSamePage).toBe(true)
+  })
+})