-
Notifications
You must be signed in to change notification settings - Fork 122
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
Relationship to private security activities #2
Conversation
👍 |
LGTM |
Is there concern that people won't read that and assume that this repo is where they should report security issues? Might there be a better name for the repo/wg? |
I should clarify that my questions above are not objections. This can totally land if it's all LGTMs and stuff. If anyone does want to consider a different name, perhaps something like "security policy wg" might be slightly better than just "security wg" even if it doesn't cover everything the wg will do. |
I think Rich has a good point. If the first line is "dont report security issues here" the naming could probably be improved. Never the less, wording is LGTM. |
LGTM with rename to "Security policy wg" |
This is landable, but I don't have access rights to do so. I annotated with the landing meta-data: https://github.com/sam-github/security-wg/commit/91f263c3a3d8a5a86d0e66a61fe9bba9b552b131 @rvagg can you land? |
PR-URL: #2 Reviewed-By: Сковорода Никита Андреевич <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Johan Bergström <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Landed in 28ae71a |
PR-URL: nodejs/security-wg#2 Reviewed-By: Сковорода Никита Андреевич <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Johan Bergström <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
PR-URL: nodejs/security-wg#2 Reviewed-By: Сковорода Никита Андреевич <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Johan Bergström <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Proposed relationship and wording for that relationship. Comments?
/cc @nodejs/security