-
Notifications
You must be signed in to change notification settings - Fork 5
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
OQS security response process #124
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
ffad71c
to
746fb6e
Compare
Co-authored-by: Douglas Stebila <[email protected]> Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
I think this can be marked as "Ready for review" and proceed with getting final reviews towards merging. |
* Add security announcement mailing list Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
@open-quantum-safe/tsc @open-quantum-safe/security-managers I think that all discussions have now been resolved satisfactorily. Please review the latest version so we can get this merged! |
Signed-off-by: Spencer Wilson <[email protected]>
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.
See additional single comments and apologies for not wrapping them into one coherent review - I simply didn't assume there'd still be so many items in need of clarification :-(
Signed-off-by: Spencer Wilson <[email protected]>
The one-liners are addressed in the latest commit; others are addressed in the relevant threads. |
I think most members of the proposed Vulnerability Management Team has commented on this, but tagging those who haven't yet commented just in case: @bhess, @praveksharma |
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.
Thanks for this work @SWilson4! See the single comment inline.
Signed-off-by: Spencer Wilson <[email protected]>
@baentsch @bhess @brian-jarvis-aws Your feedback is (hopefully) addressed in the latest commit. |
The more I think about my recent commentary/concerns, and also taking into account security notification registrations by commercial product organizations (another one just arrived in our Inboxes), what about the proposal to ask those organizations whether they'd be willing to provide personnel for the VMT? This would make it more resilient and align OQS external and internal interests. Topic for discussion next Tuesday @dstebila ? |
Sure, we can send an email to the new security announcements list (once it's populated) asking for volunteers from there interested in being more active in OQS security responses. But I don't think a nything needs to change in this PR for us to do that. |
This PR proposes a security response process for OQS. Please read it over and provide feedback. I expect it will undergo a fair bit of revision, but hopefully this gives us a starting point.
I have left a number of comments requesting feedback on specific points throughout the document. These are clearly marked by COMMENT. I'll keep the PR in draft until all of these are removed.
Closes #60.