Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

@vercel/next matches exact route match for dynamic pages #16076

Closed
Timer opened this issue Aug 11, 2020 · 2 comments
Closed

@vercel/next matches exact route match for dynamic pages #16076

Timer opened this issue Aug 11, 2020 · 2 comments
Assignees
Labels
Navigation Related to Next.js linking (e.g., <Link>) and navigation.

Comments

@Timer
Copy link
Member

Timer commented Aug 11, 2020

Bug report

Describe the bug

@vercel/next matches exact paths for dynamic routes, successfully invoking a lambda when the page is meant to be static.

To Reproduce

View success instead of 404 here.

https://test-direct-access.vercel.app/_src

❯ httpstat https://test-direct-access.vercel.app/test/[slug]

Connected to 76.76.21.21:443

HTTP/2.0 200 OK
Server: Vercel
Age: 0
Cache-Control: public, max-age=0, must-revalidate
Content-Type: text/html; charset=utf-8
Date: Tue, 11 Aug 2020 14:05:05 GMT
Strict-Transport-Security: max-age=63072000; includeSubDomains; preload
X-Nextjs-Page: /test/[slug]
X-Vercel-Cache: MISS
X-Vercel-Id: cle1::sfo1::lhkc7-1597154704606-04600048d451

Body discarded

  DNS Lookup   TCP Connection   TLS Handshake   Server Processing   Content Transfer
[      1ms  |          36ms  |        196ms  |            811ms  |             6ms  ]
            |                |               |                   |                  |
   namelookup:1ms            |               |                   |                  |
                       connect:38ms          |                   |                  |
                                   pretransfer:235ms             |                  |
                                                     starttransfer:1047ms           |
                                                                                total:1053ms  

Expected behavior

The page should 404.

@Timer Timer added this to the 9.x.x milestone Aug 11, 2020
@styfle styfle modified the milestones: 11.x.x, 12.0.4 Nov 5, 2021
@timneutkens timneutkens added the Navigation Related to Next.js linking (e.g., <Link>) and navigation. label Nov 16, 2021
@timneutkens timneutkens removed this from the 12.0.5 milestone Nov 17, 2021
@ijjk
Copy link
Member

ijjk commented Feb 3, 2022

Closing per #33808

@ijjk ijjk closed this as completed Feb 3, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2022

This closed issue has been automatically locked because it had no new activity for a month. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Navigation Related to Next.js linking (e.g., <Link>) and navigation.
Projects
None yet
Development

No branches or pull requests

4 participants