-
-
Notifications
You must be signed in to change notification settings - Fork 865
Add PSRT coordination process and messaging templates #1348
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
Changes from all commits
e388822
e200e2d
ea55f7a
0344b7c
8e23761
18ac764
4c3dc6a
09a831c
2211cdd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,3 +14,4 @@ Development workflow | |
grammar | ||
porting | ||
sbom | ||
psrt |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,160 @@ | ||
Python Security Response Team (PSRT) | ||
==================================== | ||
|
||
The Python Security Response Team (PSRT) is responsible for handling | ||
vulnerability reports for CPython and pip. | ||
|
||
Vulnerability report triage | ||
--------------------------- | ||
|
||
Vulnerability reports are sent to one of two locations, | ||
the long-standing ``security@python.org`` mailing list | ||
or using the private vulnerability reporting feature | ||
of GitHub Security Advisories (GHSA). | ||
|
||
For reports sent to ``security@python.org``, a PSRT admin | ||
will triage the report and if the report seems plausible | ||
(that is, not spam and for the correct project) will reply with | ||
instructions on how to report the vulnerability on GitHub. | ||
|
||
If the reporter doesn't want to use GitHub's Security Advisories feature | ||
then the PSRT admins can create a draft report on behalf of the reporter. | ||
|
||
Coordinating a vulnerability report | ||
----------------------------------- | ||
|
||
Each report will have a member of the PSRT assigned as the "coordinator". | ||
The coordinator will be responsible for following the below process and | ||
will be publicly credited on vulnerability records post-publication. | ||
|
||
If a coordinator can't complete the process for any reason (time obligation, | ||
vacation, etc.) they must find a replacement coordinator in the PSRT | ||
and reassign the vulnerability report appropriately. | ||
|
||
Coordinators are expected to collaborate with other PSRT members and core developers | ||
when needed for guidance on whether the report is an actual vulnerability, | ||
severity, advisory text, and fixes. | ||
|
||
**The vulnerability coordination process is:** | ||
|
||
* Coordinator will determine whether the report constitutes a vulnerability. If the report isn't a vulnerability, | ||
the reporter should be notified appropriately. Close the GHSA report, the report can be reopened if | ||
sufficient evidence is later obtained that the report is a vulnerability. | ||
|
||
* After a vulnerability report is accepted, a Common Vulnerabilities and Exposures (CVE) ID must be assigned. If this is not done | ||
automatically, then a CVE ID can be obtained by the coordinator sending an email to ``cna@python.org``. | ||
No details about the vulnerability report need to be shared with the PSF CVE Numbering Authority (CNA) for a CVE ID to be reserved. | ||
|
||
* If the report is a vulnerability, the coordinator will determine the severity of the vulnerability. Severity is one of: | ||
**Low**, **Medium**, **High**, and **Critical**. Coordinators can use their knowledge of the code, how the code is likely used, | ||
or another mechanism like Common Vulnerability Scoring System (CVSS) for determining a severity. Add this information to the GitHub Security Advisory. | ||
|
||
* Once a CVE ID is assigned, the coordinator will share the acceptance and CVE ID with the reporter. | ||
Use this CVE ID for referencing the vulnerability. The coordinator will ask the reporter | ||
if the reporter would like to be credited publicly for the report and if so, how they would like to be credited. | ||
Add this information to the GitHub Security Advisory. | ||
|
||
* The coordinator authors the vulnerability advisory text. The advisory must include the following information: | ||
|
||
* Title should be a brief description of the vulnerability and affected component | ||
(for example, "Buffer over-read in SSLContext.set_npn_protocols()") | ||
|
||
* Short description of the vulnerability, impact, and the conditions where the affected component is vulnerable, if applicable. | ||
|
||
* Affected versions. This could be "all versions", but if the vulnerability exists in a new feature | ||
or removed feature then this could be different. Include versions that are end-of-life in this calculation | ||
(for example, "Python 3.9 and earlier", "Python 3.10 and later", "all versions of Python"). | ||
|
||
* Affected components and APIs. The module, function, class, or method must be specified so users can | ||
search their codebase for usage. For issues affecting the entire project, this can be omitted. | ||
|
||
* Mitigations for the vulnerability beyond upgrading to a patched version, if applicable. | ||
|
||
This can all be done within the GitHub Security Advisory UI for easier collaboration between reporter and coordinator. | ||
|
||
* The coordinator determines the fix approach and who will provide a patch. | ||
Some reporters are willing to provide or collaborate to create a patch, | ||
otherwise relevant core developers can be invited to collaborate by | ||
the coordinator. | ||
|
||
* For **Low** and **Medium** severity vulnerabilities it is acceptable | ||
to develop a patch in public. | ||
The pull request must be marked with the ``security`` and ``release-blocker`` | ||
labels so that a release is not created without including the patch. | ||
|
||
* For **High** and **Critical** severity vulnerabilities the patch must be | ||
developed privately using GitHub Security Advisories' "Private Forks" feature. | ||
Core developers can be added to the GitHub Security Advisory via "collaborators" | ||
to work on the fix together. Once a patch is approved privately and tested, | ||
a public issue and pull request can be created with | ||
the ``security`` and ``release-blocker`` labels. | ||
|
||
* Once the pull request is merged the advisory can be published. The coordinator will send the advisory by email | ||
to ``security-announce@python.org`` using the below template. Backport labels must be added as appropriate. | ||
After the advisory is published a CVE record can be created. | ||
|
||
Template responses | ||
------------------ | ||
|
||
sethmlarson marked this conversation as resolved.
Show resolved
Hide resolved
|
||
These template responses should be used as guidance for messaging | ||
in various points in the process above. They are not required to be sent as-is, | ||
please feel free to adapt them as needed for the current context. | ||
|
||
**Directing to GitHub Security Advisories:** | ||
|
||
.. highlight:: none | ||
|
||
:: | ||
|
||
Thanks for submitting this report. | ||
We use GitHub Security Advisories for triaging vulnerability reports, | ||
are you able to submit your report directly to GitHub? | ||
|
||
https://github.com/python/cpython/security/advisories/new | ||
|
||
If you're unable to submit a report to GitHub (due to not having a GitHub | ||
account or something else) let me know and I will create a GitHub Security | ||
Advisory on your behalf, although you won't be able to participate directly | ||
in discussions. | ||
|
||
**Rejecting a vulnerability report:** | ||
|
||
:: | ||
|
||
Thanks for your report. We've determined that the report doesn't constitute | ||
a vulnerability. Let us know if you disagree with this determination. | ||
If you are interested in working on this further, you can optionally open a | ||
public issue on GitHub. | ||
|
||
**Accepting a vulnerability report:** | ||
|
||
:: | ||
|
||
Thanks for your report. We've determined that the report | ||
is a vulnerability. We've assigned {CVE-YYYY-XXXX} and determined | ||
a severity of {Low,Medium,High,Critical}. Let us know if you disagree | ||
with the determined severity. | ||
|
||
If you would like to be publicly credited for this vulnerability as the | ||
reporter, please indicate that, along with how you would like to be | ||
credited (name or organization). | ||
|
||
Please keep this vulnerability report private until we've published | ||
an advisory to ``security-announce@python.org``. | ||
|
||
**Advisory email:** | ||
|
||
:: | ||
|
||
Title: [{CVE-YYYY-XXXX}] {title} | ||
|
||
There is a {LOW, MEDIUM, HIGH, CRITICAL} severity vulnerability | ||
affecting {project}. | ||
|
||
{description} | ||
|
||
Please see the linked CVE ID for the latest information on | ||
affected versions: | ||
|
||
* https://www.cve.org/CVERecord?id={CVE-YYYY-XXXX} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This link won't work on initial disclosure, right? Or do they activate that quickly? Agreed with keeping it in the initial email though, since these get publicly archived and essentially are our disclosure mechanism. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It becomes public almost instantly after it's pushed, but yes it requires a manual publication from the PSF CNA. |
||
* {pull request URL} |
Uh oh!
There was an error while loading. Please reload this page.