-
Notifications
You must be signed in to change notification settings - Fork 124
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
Incoming request URL is not updated to include dynamic route params #35
Comments
I got bitten by this. I thought my dynamic routes would work as is with this setup, but apparently not. |
I had success with the above. |
Unfortunately, I just have pages which have dynamic routes - and those should be given in "params" to the functions. Some pointers to help fix this... The
Hence this part in the actual worker: At least, that is my guess what is sufficient. |
Apparently this is closer to my problem, and much closer also to the correct fix: #32 |
Next.js attaches dynamic route params as query params. In normal routes, developers usually grab these values from the
params
object:But in Edge API routes, there is no such mechanism. In order to grab these values, you have to look at the incoming request URL:
We should pass these handling functions the finalized URL which includes any dynamic route params in the query string.
The text was updated successfully, but these errors were encountered: