-
-
Notifications
You must be signed in to change notification settings - Fork 996
Adds SECURITY.md to outline our security policies #2086
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?
Conversation
from a member of the website working group within 3 working days. After that, | ||
the website working group will begin their analysis. Depending on the action | ||
to be taken, you may receive followup emails. It can take several weeks before | ||
the website working group comes to a conclusion and resolve the issue. |
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.
I have mostly just copied from Django's security policy. We might want to update.
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.
I'd remove or extend the 3 working days timeline, we don't need to set a timeline on ourselves like for the framework.
- Confirm the problem. | ||
- Audit code to find any potential similar problems. | ||
- Apply the relevant patches to the codebase. | ||
- Deploy the fixed codebase. |
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.
Do we know how we want to disclose a security issue, since for the website, we don't really need to make releases and ask users to update. Maybe this section should be titled differently. Thoughts?
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.
My first thought would be, do we even need a disclosure policy at all?
to follow few guidelines that helps us in analysis and resolving the issue quicker. | ||
|
||
- Include a runnable proof of concept to reproduce the issue | ||
- User input must be sanitized |
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.
Are there other website specific guidelines we want to add?
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.
What does "User input must be sanitized" means in this context?
I'm not sure what it means, so it could also be confusing for a reporter.
As discussed in our last meeting, along with a
.well-known/security.txt
, it's nice to have aSECURITY.md
to be shown as our security policy in the GitHub Security tab.This PR adds the policies for reporting a bug and disclosing.