Add SECURITY.md#585
Conversation
pradyunsg
left a comment
There was a problem hiding this comment.
This should live in the documentation as the "security policy" page.
You'll probably want to note that security issues should not be filed publicly and sent via email to the developers/maintainers (ideally by providing an email here).
The language of the "reporting a bug" seems to be more in line with something for the contributing guide.
Further, it references the use of email and a security team, which haven't been mentioned in the rest of the document.
I'm not sure I agree. The security policy for the project should be easily noticeable and not some page hidden away in the docs.
Agreed, I'll add this in. I've also mentioned the need for an email to receive reports in the first comment.
Well, it's the standard format followed by platforms like HackerOne, for reports. I figured it'd fit in here as well. What do you suggest?
Yeah, again, we'll need an email from Junction's maintainers. /cc @zerothabhishek |
Would the way Django handles their security policy serve as enough prior art to justify my position? PyPI also has a section in their documentation, that links to a page that's rendered at https://pypi.org/security. Neither project stores it the root of the project's repository.
Something like what Django does, should be good enough for us:
I think it is also important to note that given that this project does not have any active maintainers, it's probably better to not promise a timeline for a response to the reporter. |
Fair enough. In that case, I'll close this PR and you can proceed to add the security policy over at #577.
LGTM.
Let's wait on @zerothabhishek for a final take. |
|
@palnabarun We should create a email alias via magudi for reporting security issues. Then we can proceed with the documentation. Can you keep it as an action item when you update the PyCon 2020 email aliases in magudi ? |
|
Sure. 👍 |
|
Let's go ahead with webmaster@in.pycon.org which I believe exists already. @pradyunsg You may want to add a |
|
Closing since #584 is the tracking issue and we're going to be taking a different approach for documenting our security policy (in the built documentation vs as a security.md file). |
Fixes #584
This PR is up for debate/discussion. What else needs to be added to this?
Also, we need an email to receive reports.