Skip to content

docs(security): add secondary contact/security escalation policy#6315

Merged
Uzlopak merged 5 commits intofastify:mainfrom
UlisesGascon:patch-1
Sep 18, 2025
Merged

docs(security): add secondary contact/security escalation policy#6315
Uzlopak merged 5 commits intofastify:mainfrom
UlisesGascon:patch-1

Conversation

@UlisesGascon
Copy link
Contributor

Signed-off-by: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Copy link
Contributor

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or else?

@UlisesGascon
Copy link
Contributor Author

Or else?

I’ll add clarification. Escalation means contacting the OpenJS Foundation CNA if the project team is unresponsive. The CNA ensures the report is acknowledged, helps coordinate disclosure, and can assign a CVE if necessary. It’s not a bypass or a guarantee of a faster fix. It is just a safeguard to ensure reports are handled properly.

This change is also being adopted by other projects in the foundation (e.g. nodejs/node#59806, jquery/jquery#5701....), so we’ll have a consistent approach across OpenJS as discussed on the CPC Meeting.

@Uzlopak
Copy link
Contributor

Uzlopak commented Sep 15, 2025

@UlisesGascon You answered in the meantime, but below was what i wanted to answer you before your post.

First of all, an escalation should be imho not the first thing I should read. It should be at the bottom of the document, after we described how we handle security reports.

We have self mandated 4 day reaction time.

Also i think it is critical to not clarify what the scope of the escalation is. What is here the role of the OpenJS Foundation?
What is ultima ratio of the escalation? Keeping potential project funds? Project Takeover by the Foundation? Creating a PR which fastify-maintainers are obligated to merge?

Or is the wording "Escalation" the issue? Is it more about that the OpenJS Foundation CNA will then separately will publish a CVE? Then the wording as "escalation" is inappropriate.

Without any clarification I think we should reject this policy change.

@Uzlopak
Copy link
Contributor

Uzlopak commented Sep 15, 2025

I see, that also in the nodejs pr the "escalation" was needing clarification. I think we should encourage that the OpenJS Foundation changes the template for the sake of clarity.

UlisesGascon and others added 2 commits September 15, 2025 17:09
Co-authored-by: Aras Abbasi <aras.abbasi@googlemail.com>
Signed-off-by: Ulises Gascón <ulisesgascongonzalez@gmail.com>
@UlisesGascon
Copy link
Contributor Author

Thanks for the feedback @Uzlopak! I will bring this discussion to the Security Collab Space to re-define the policy/template. In the meantime I relocated the text in ac96ec2 with your suggestions included.

Copy link
Contributor

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your work

@Fdawgs Fdawgs changed the title docs: add security escalation policy docs(security): add security escalation policy Sep 15, 2025
Co-authored-by: Frazer Smith <frazer.dev@icloud.com>
Signed-off-by: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Copy link
Contributor

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@metcoder95 metcoder95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just linting seems failing

@Uzlopak
Copy link
Contributor

Uzlopak commented Sep 16, 2025

@fastify/leads

Do you agree with this PR?

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Member

@gurgunday gurgunday left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Uzlopak Uzlopak merged commit a4fe041 into fastify:main Sep 18, 2025
5 checks passed
@Uzlopak Uzlopak changed the title docs(security): add security escalation policy docs(security): add secondary contact/security escalation policy Sep 18, 2025
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.

7 participants