-
Notifications
You must be signed in to change notification settings - Fork 86
[Bug]: Edge Functions Beta - Middleware rewrites not working as expected #1321
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
Comments
Thanks for the quick fix for this! Unfortunately I've found another possible bug related to this. Whenever I go to the dynamic AB test route the edge function goes into an endless loop (4 times a second) with the following logs:
NOTE: I reverted my deploy to protect my quota but anyone should be able to reproduce the bug from my git repo. I'd like to try and drop next middleware for pure edge functions as recommended, but it will require some time testing to see how it behaves with prefetching via next/Link. Thanks again! |
Hi @JohnGemstone , Thanks for the follow up bug report (and an even bigger thank you for the initial reproduction project, was extremely helpful)! @ascorbic has just merged a fix for the infinite reload loop that you mentioned as part of #1325 . Hopefully that sorts things out, but if there's any other bugs that come out of the woodwork, don't hesitate to reach out again. |
No problem, thanks for getting the fixes across so quickly! |
Summary
I've updated my middleware AB testing repo to test out the Edge Functions Beta in 4.4.2 and came across these following bugs. Comparisons are with the same repo but with the Edge Functions environment variable set to true or false respectively.
Edge functions ON
Edge functions OFF (deployed here)
Steps to reproduce
Go to my repo and choose either standard or dynamic routes.
dynamic routes:
original
rename totest
next/link
nav menustandard routes:
original
rename totest
next/link
nav menuA link to a reproduction repository
https://github.com/JohnGemstone/abtest-netlify-next
Plugin version
4.4.2
More information about your build
netlify.toml
)What OS are you using?
Mac OS
Your netlify.toml file
`netlify.toml`
Builds logs (or link to your logs)
Build logs
The text was updated successfully, but these errors were encountered: