-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
chore: add skew-version support in Cloudflare open-next worker #7980
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
base: main
Are you sure you want to change the base?
chore: add skew-version support in Cloudflare open-next worker #7980
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## main #7980 +/- ##
==========================================
- Coverage 73.10% 73.09% -0.02%
==========================================
Files 95 95
Lines 8355 8354 -1
Branches 219 219
==========================================
- Hits 6108 6106 -2
- Misses 2246 2247 +1
Partials 1 1 ☔ View full report in Codecov by Sentry. |
again! is this something that is happening quite often? 😓 |
Yea, it happens pretty often now that I think about it. |
oof... and do you know if it is just happening on the open-next build? |
ok, so sorry for the inconvenience 🙇 I'll look into addressing the issue then 🙇 |
No problem! It's not a required check so it's not breaking anything |
good! but it must be pretty annoying 😅😓 |
3f20cb6
to
9256c7a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGMT !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Buit can we document this variable for the time being somewhere? (the OPEN_NEXT_CLOUDFLARE)
Every time you write LGMT I feel like I want to eat Subway's BLT for some reason |
This comment was marked as off-topic.
This comment was marked as off-topic.
Looks good, me too! |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry @avivkeller let's wait for tomorrow to merge this if that's ok 🙏 It should be all good but before merging I wanted to double check that the deployed version does implement version skew protection correctly, I'll do that at some point tomorrow 🙇 |
Sorry, I'm having some trouble validating that the skew protection works correctly (since it seems like it (intentionally) does not work on |
Description
This PR adds support for skew protection in the Cloudflare open-next deployment (which has been recently added to the open-next adapter (in 1.5.0))
For more details on the feature see: https://opennext.js.org/cloudflare/howtos/skew
Validation
I've applied the changes and deployed the site: https://nodejs-org.testing.devprod.cloudflare.dev/en
And everything seems to be working as expected 🙂
I'll double check with @vicb if he has some suggestion to best test that the skew protection is actually in place and working as expected 🙂
Related Issues
Check List
pnpm format
to ensure the code follows the style guide.pnpm test
to check if all tests are passing.pnpm build
to check if the website builds without errors.