Rename to resolvedUrl and add more tests

docs/basic-features/data-fetching.md

@@ -543,7 +543,7 @@ The `context` parameter is an object containing the following keys:
 - `query`: The query string.
 - `preview`: `preview` is `true` if the page is in the preview mode and `false` otherwise. See the [Preview Mode documentation](/docs/advanced-features/preview-mode.md).
 - `previewData`: The preview data set by `setPreviewData`. See the [Preview Mode documentation](/docs/advanced-features/preview-mode.md).
-- `asPath`: A normalized version of the request URL that strips the `_next/data` prefix for client transitions which matches the `asPath` value provided to the router.
+- `resolvedUrl`: A normalized version of the request URL that strips the `_next/data` prefix for client transitions and includes original query values.
 
 > **Note**: You can import modules in top-level scope for use in `getServerSideProps`.
 > Imports used in `getServerSideProps` will not be bundled for the client-side.

packages/next/build/webpack/loaders/next-serverless-loader.ts

@@ -496,7 +496,7 @@ const nextServerlessLoader: loader.Loader = function () {
           !fromExport &&
           (getStaticProps || getServerSideProps)
         ) {
-          const curQuery = {...parsedUrl.query}
+          const origQuery = parseUrl(req.url, true).query
 
           ${
             pageIsDynamicRoute
@@ -507,7 +507,7 @@ const nextServerlessLoader: loader.Loader = function () {
                 delete parsedUrl.search
 
                 for (const param of Object.keys(defaultRouteRegex.groups)) {
-                  delete curQuery[param]
+                  delete origQuery[param]
                 }
               }
             `
@@ -517,7 +517,7 @@ const nextServerlessLoader: loader.Loader = function () {
           parsedUrl.pathname = denormalizePagePath(parsedUrl.pathname)
           renderOpts.normalizedAsPath = formatUrl({
             ...parsedUrl,
-            query: curQuery
+            query: origQuery
           })
         }
 

packages/next/next-server/server/next-server.ts

@@ -1111,13 +1111,11 @@ export default class Server {
             ...components,
             ...opts,
             isDataReq,
-            normalizedAsPath: isDataReq
-              ? formatUrl({
-                  pathname: urlPathname,
-                  // make sure to only add query values from original URL
-                  query: parseUrl(req.url!, true).query,
-                })
-              : undefined,
+            normalizedAsPath: formatUrl({
+              pathname: urlPathname,
+              // make sure to only add query values from original URL
+              query: parseUrl(req.url!, true).query,
+            }),
           }
 
           renderResult = await renderToHTML(

packages/next/next-server/server/render.tsx

@@ -689,7 +689,7 @@ export async function renderToHTML(
           req,
           res,
           query,
-          asPath,
+          resolvedUrl: asPath,
           ...(pageIsDynamic ? { params: params as ParsedUrlQuery } : undefined),
           ...(previewData !== false
             ? { preview: true, previewData: previewData }

packages/next/types/index.d.ts

@@ -120,7 +120,7 @@ export type GetServerSidePropsContext<
   query: ParsedUrlQuery
   preview?: boolean
   previewData?: any
-  asPath: string
+  resolvedUrl: string
 }
 
 export type GetServerSidePropsResult<P> = {

test/integration/getserversideprops/next.config.js

@@ -0,0 +1,19 @@
+module.exports = {
+  // replace me
+  async rewrites() {
+    return [
+      {
+        source: '/blog-post-1',
+        destination: '/blog/post-1',
+      },
+      {
+        source: '/blog-post-2',
+        destination: '/blog/post-2?hello=world',
+      },
+      {
+        source: '/blog-:param',
+        destination: '/blog/post-3',
+      },
+    ]
+  },
+}

test/integration/getserversideprops/pages/blog/[post]/index.js

@@ -2,7 +2,7 @@ import React from 'react'
 import Link from 'next/link'
 import { useRouter } from 'next/router'
 
-export async function getServerSideProps({ params, asPath }) {
+export async function getServerSideProps({ params, resolvedUrl }) {
   if (params.post === 'post-10') {
     await new Promise((resolve) => {
       setTimeout(() => resolve(), 1000)
@@ -16,14 +16,14 @@ export async function getServerSideProps({ params, asPath }) {
   return {
     props: {
       params,
-      gsspAsPath: asPath,
+      resolvedUrl,
       post: params.post,
       time: (await import('perf_hooks')).performance.now(),
     },
   }
 }
 
-export default ({ post, time, params, appProps, gsspAsPath }) => {
+export default ({ post, time, params, appProps, resolvedUrl }) => {
   return (
     <>
       <p>Post: {post}</p>
@@ -32,7 +32,7 @@ export default ({ post, time, params, appProps, gsspAsPath }) => {
       <div id="query">{JSON.stringify(useRouter().query)}</div>
       <div id="app-query">{JSON.stringify(appProps.query)}</div>
       <div id="app-url">{appProps.url}</div>
-      <div id="gssp-asPath">{gsspAsPath}</div>
+      <div id="resolved-url">{resolvedUrl}</div>
       <Link href="/">
         <a id="home">to home</a>
       </Link>

test/integration/getserversideprops/pages/something.js

@@ -2,10 +2,10 @@ import React from 'react'
 import Link from 'next/link'
 import { useRouter } from 'next/router'
 
-export async function getServerSideProps({ params, query, asPath }) {
+export async function getServerSideProps({ params, query, resolvedUrl }) {
   return {
     props: {
-      gsspAsPath: asPath,
+      resolvedUrl: resolvedUrl,
       world: 'world',
       query: query || {},
       params: params || {},
@@ -22,7 +22,7 @@ export default ({
   random,
   query,
   appProps,
-  gsspAsPath,
+  resolvedUrl,
 }) => {
   return (
     <>
@@ -34,7 +34,7 @@ export default ({
       <div id="query">{JSON.stringify(useRouter().query)}</div>
       <div id="app-query">{JSON.stringify(appProps.query)}</div>
       <div id="app-url">{appProps.url}</div>
-      <div id="gssp-asPath">{gsspAsPath}</div>
+      <div id="resolved-url">{resolvedUrl}</div>
       <Link href="/">
         <a id="home">to home</a>
       </Link>

test/integration/getserversideprops/test/index.test.js

@@ -7,6 +7,7 @@ import {
   check,
   fetchViaHTTP,
   findPort,
+  File,
   getBrowserBodyText,
   getRedboxHeader,
   killApp,
@@ -22,7 +23,8 @@ import { join } from 'path'
 
 jest.setTimeout(1000 * 60 * 2)
 const appDir = join(__dirname, '..')
-const nextConfig = join(appDir, 'next.config.js')
+const nextConfig = new File(join(appDir, 'next.config.js'))
+
 let app
 let appPort
 let buildId
@@ -297,7 +299,7 @@ const runTests = (dev = false) => {
     const data = await renderViaHTTP(appPort, curUrl)
     const { appProps, pageProps } = JSON.parse(data)
 
-    expect(pageProps.gsspAsPath).toEqual('/blog/post-1')
+    expect(pageProps.resolvedUrl).toEqual('/blog/post-1')
 
     expect(appProps).toEqual({
       url: curUrl,
@@ -307,12 +309,27 @@ const runTests = (dev = false) => {
     })
   })
 
+  it('should have original req.url for /_next/data request dynamic page with query', async () => {
+    const curUrl = `/_next/data/${buildId}/blog/post-1.json`
+    const data = await renderViaHTTP(appPort, curUrl, { hello: 'world' })
+    const { appProps, pageProps } = JSON.parse(data)
+
+    expect(pageProps.resolvedUrl).toEqual('/blog/post-1?hello=world')
+
+    expect(appProps).toEqual({
+      url: curUrl + '?hello=world',
+      query: { post: 'post-1', hello: 'world' },
+      asPath: '/blog/post-1?hello=world',
+      pathname: '/blog/[post]',
+    })
+  })
+
   it('should have original req.url for /_next/data request', async () => {
     const curUrl = `/_next/data/${buildId}/something.json`
     const data = await renderViaHTTP(appPort, curUrl)
     const { appProps, pageProps } = JSON.parse(data)
 
-    expect(pageProps.gsspAsPath).toEqual('/something')
+    expect(pageProps.resolvedUrl).toEqual('/something')
 
     expect(appProps).toEqual({
       url: curUrl,
@@ -322,20 +339,78 @@ const runTests = (dev = false) => {
     })
   })
 
+  it('should have original req.url for /_next/data request with query', async () => {
+    const curUrl = `/_next/data/${buildId}/something.json`
+    const data = await renderViaHTTP(appPort, curUrl, { hello: 'world' })
+    const { appProps, pageProps } = JSON.parse(data)
+
+    expect(pageProps.resolvedUrl).toEqual('/something?hello=world')
+
+    expect(appProps).toEqual({
+      url: curUrl + '?hello=world',
+      query: { hello: 'world' },
+      asPath: '/something?hello=world',
+      pathname: '/something',
+    })
+  })
+
   it('should have correct req.url and query for direct visit dynamic page', async () => {
     const html = await renderViaHTTP(appPort, '/blog/post-1')
     const $ = cheerio.load(html)
     expect($('#app-url').text()).toContain('/blog/post-1')
     expect(JSON.parse($('#app-query').text())).toEqual({ post: 'post-1' })
-    expect($('#gssp-asPath').text()).toBe('/blog/post-1')
+    expect($('#resolved-url').text()).toBe('/blog/post-1')
+  })
+
+  it('should have correct req.url and query for direct visit dynamic page rewrite direct', async () => {
+    const html = await renderViaHTTP(appPort, '/blog-post-1')
+    const $ = cheerio.load(html)
+    expect($('#app-url').text()).toContain('/blog-post-1')
+    expect(JSON.parse($('#app-query').text())).toEqual({ post: 'post-1' })
+    expect($('#resolved-url').text()).toBe('/blog/post-1')
+  })
+
+  it('should have correct req.url and query for direct visit dynamic page rewrite direct with internal query', async () => {
+    const html = await renderViaHTTP(appPort, '/blog-post-2')
+    const $ = cheerio.load(html)
+    expect($('#app-url').text()).toContain('/blog-post-2')
+    expect(JSON.parse($('#app-query').text())).toEqual({
+      post: 'post-2',
+      hello: 'world',
+    })
+    expect($('#resolved-url').text()).toBe('/blog/post-2')
+  })
+
+  it('should have correct req.url and query for direct visit dynamic page rewrite param', async () => {
+    const html = await renderViaHTTP(appPort, '/blog-post-3')
+    const $ = cheerio.load(html)
+    expect($('#app-url').text()).toContain('/blog-post-3')
+    expect(JSON.parse($('#app-query').text())).toEqual({
+      post: 'post-3',
+      param: 'post-3',
+    })
+    expect($('#resolved-url').text()).toBe('/blog/post-3')
+  })
+
+  it('should have correct req.url and query for direct visit dynamic page with query', async () => {
+    const html = await renderViaHTTP(appPort, '/blog/post-1', {
+      hello: 'world',
+    })
+    const $ = cheerio.load(html)
+    expect($('#app-url').text()).toContain('/blog/post-1?hello=world')
+    expect(JSON.parse($('#app-query').text())).toEqual({
+      post: 'post-1',
+      hello: 'world',
+    })
+    expect($('#resolved-url').text()).toBe('/blog/post-1?hello=world')
   })
 
   it('should have correct req.url and query for direct visit', async () => {
     const html = await renderViaHTTP(appPort, '/something')
     const $ = cheerio.load(html)
     expect($('#app-url').text()).toContain('/something')
     expect(JSON.parse($('#app-query').text())).toEqual({})
-    expect($('#gssp-asPath').text()).toBe('/something')
+    expect($('#resolved-url').text()).toBe('/something')
   })
 
   it('should return data correctly', async () => {
@@ -593,10 +668,9 @@ describe('getServerSideProps', () => {
 
   describe('serverless mode', () => {
     beforeAll(async () => {
-      await fs.writeFile(
-        nextConfig,
-        `module.exports = { target: 'serverless' }`,
-        'utf8'
+      await nextConfig.replace(
+        '// replace me',
+        `target: 'experimental-serverless-trace', `
       )
       await nextBuild(appDir)
       stderr = ''
@@ -608,14 +682,16 @@ describe('getServerSideProps', () => {
       })
       buildId = await fs.readFile(join(appDir, '.next/BUILD_ID'), 'utf8')
     })
-    afterAll(() => killApp(app))
+    afterAll(async () => {
+      await killApp(app)
+      nextConfig.restore()
+    })
 
     runTests()
   })
 
   describe('production mode', () => {
     beforeAll(async () => {
-      await fs.remove(nextConfig)
       await nextBuild(appDir, [], { stdout: true })
 
       appPort = await findPort()