-
Notifications
You must be signed in to change notification settings - Fork 21
add security and privacy considerations living document #55
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
Conversation
|
initial draft based on discussion here: #45 will update it based on relevant discussions in the thread, as well as from TPAC before marking it as ready to merge. |
khushalsagar
left a comment
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.
Looking good!
victorhuangwq
left a comment
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 the review. will make changes accordingly, alongside the updates that I will be making to include points from TPAC and the current issue thread.
…Attacks on Tool Implementations" and clarify associated risks and examples
…urity considerations
…te tracking in security considerations
…iderations document
|
Left a comment for one attack vector that deserves more thinking and can be deferred to another PR. LGTM otherwise! |
|
@khushalsagar @bwalderman @anssiko PR is ready to be merged! Could I get either of your help to merge it? |
No description provided.