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

[react] Old React versions are not EOL (security updates) #6684

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

istvanp
Copy link

@istvanp istvanp commented Jan 31, 2025

The previous update to React #6675 does not properly reflect React's stance on security updates.

See React maintainer @gaearon's comment: reactjs/react.dev#1745 (comment)

If there was a critical security vulnerability affecting all majors we would do every effort to release a patch for the latest minor of every major, as well as for every minor of the current major.

I think the wording is clear that versions 18 and below would receive a security update if there was a critical vulnerability.

I don't think there is a new stance on this from React but I could be wrong (the official comment is from 2021).

See React maintainer @gaearon's comment: reactjs/react.dev#1745 (comment)

> If there was a critical security vulnerability affecting all majors we would do every effort to release a patch for the latest minor of every major, as well as for every minor of the current major.
Copy link

welcome bot commented Jan 31, 2025

Thank you for opening this pull request 👍. If you are not familiar with the project, please check out our Contributing Guidelines and our Guiding Principles. Also take a look at our Hacking Guide if you intend to work on site internals.

@usta
Copy link
Member

usta commented Jan 31, 2025

but you are missing the main point , those are eoled but if they found a critical bug and if any hotfix applied we increase the eol date to latest release date, ( you can observe this on releaseCycle: "15" )
on the other hand it is clearly mentioned that if a major version releases older version's support ends ( only exception is found critical bugs )
So my 2 cents for this PR is 👎

@usta usta changed the title Old React versions are not EOL (security updates) [react] Old React versions are not EOL (security updates) Jan 31, 2025
@istvanp
Copy link
Author

istvanp commented Jan 31, 2025

But that is why you have another column for "Security Support"? They are EOL for development only, not for critical security fixes.

That column's job (as I understand it) is to indicate if it's safe to remain on an older version or not should there be a vulnerability. If it's got a an end date it means we should move onto a newer version as soon as possible which is not the case here because they are continuing support with regards to critical security issues at any time.

@captn3m0
Copy link
Member

captn3m0 commented Feb 1, 2025

I was also surprised by #6675, since it drastically changes what we counted as security support, without any change in upstream guidance with the v19 release.

@ljharb
Copy link

ljharb commented Feb 1, 2025

Dan's comment isn't necessarily the same thing as a project-level commitment to definitely releasing security fixes, which is what I'd consider necessary for "support".

Best-effort is great! but not "supported".

@captn3m0
Copy link
Member

captn3m0 commented Feb 1, 2025

As it stands, https://endoflife.date/react has conflicting guidance in the table v/s the support text. At the very least we should update the text to inform the users about the best-effort-stance.

@pjgorman
Copy link

pjgorman commented Feb 8, 2025

A 3.5 year old issue comment does not a versioning policy make.
I've combed through the actual React versioning policy, and I find no mention of even 'best effort' to patch previous versions of major releases, nor the current Support Text indicating that "Critical Security fixes are backported to all minor releases of the current major, as well as to latest minor release of previous major releases."

Granted, I'm not doubting the React team's intent (or willingness) to backport security fixes as far as possible, but something to that end should be reflected somewhere in the project documentation.

I'd recommend updating the Support text to call it like it is:
It has been suggested that best effort will be made to address critical security vulnerabilities in previous affected major releases as well as in every minor release of the current major release, but this intent is not documented in the project's versioning policy.

Submitted an alternative PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants