Skip to content
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

Write a Privacy and Security section #166

Closed
nigelmegitt opened this issue Jul 12, 2023 · 0 comments · Fixed by #168
Closed

Write a Privacy and Security section #166

nigelmegitt opened this issue Jul 12, 2023 · 0 comments · Fixed by #168
Assignees
Labels
CR must-have Must be resolved before going to CR

Comments

@nigelmegitt
Copy link
Contributor

nigelmegitt commented Jul 12, 2023

This is mandatory. Take into account the Self-Review Questionnaire: Security and Privacy, Mitigating Browser Fingerprinting in Web Specifications, and RFC6973

See also the DAPT wiki page Considerations for the Privacy and Security Reviews.

@nigelmegitt nigelmegitt added the CR must-have Must be resolved before going to CR label Jul 12, 2023
nigelmegitt added a commit that referenced this issue Jul 19, 2023
@nigelmegitt nigelmegitt self-assigned this Jul 19, 2023
nigelmegitt added a commit that referenced this issue Jul 27, 2023
nigelmegitt added a commit that referenced this issue Jul 31, 2023
Merging this editorial change to allow the horizontal reviews to be raised. If there are further issues, please raise them separately.

Closes #166.

* Add Privacy and Security Section
* Add fingerprinting images

Address review feedback. The images themselves are slightly cryptic and don't have any link back to something that explains why they're there, so I'm not sure about the overall benefit.

* Split Privacy and Security Considerations into two

to satisfy the horizontal review requirements of the groups that review for privacy and security.

* Add note re avoiding exposing audio resources by embedding them
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CR must-have Must be resolved before going to CR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant