-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
[WIP] feat: add eol page #7990
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?
[WIP] feat: add eol page #7990
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
ran with |
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #7990 +/- ##
==========================================
- Coverage 73.13% 73.07% -0.06%
==========================================
Files 95 95
Lines 8354 8354
Branches 218 219 +1
==========================================
- Hits 6110 6105 -5
- Misses 2243 2248 +5
Partials 1 1 ☔ View full report in Codecov by Sentry. |
I feel this page should use a layout without the sidebar/metabar FYI |
this is the current pattern to expose translations to MDX, from what i can tell
@avivkeller can we use the new website for CVEs? https://www.cve.org/CVERecord?id=CVE-2025-23166 instead of cve.mitre.org? |
Also this description: "There are 4+ known vulnerabilities associated with this Node.js release. Please review their severity and details to understand the potential impact." not sure if it is fitting. I think we should lean more into what CVEs are how to understand them what having these issues are... IDK; Throwing the "Please review their severity and details to understand the potential impact." to the end-user might not be ideal. I mean in the end that's whaty they're going to do anyways, but we could use such section to explain actual concrete details of what CVEs means, what clicking these links means, idk. |
Also can the "is EoL" be "has reached EOL" or "is EOL" or just "is vulnerable" |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
I am hoping we can resolve most of the unknowns actually, via nodejs/security-wg#1501 |
Right, but let's focus on the now and not promises of the future. |
Description
Validation
EOL Page
EOL / Vulnerability Table
Link updated here
and here
Vulnerability Blog Posts
Related Issues
closes #7906
Check List
All other links on all alert boxes across the website (blog post from Matteo, Download pages, Version modal, etc) go to the /EOL page
- did not do Matteo's blog post yet - would rather someone else choose to editorialize that.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.