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

Reorganize Security and privacy considerations #216

Merged
merged 2 commits into from
Jun 5, 2023
Merged

Conversation

anssiko
Copy link
Member

@anssiko anssiko commented Jun 2, 2023

@pes10k PTAL. After another look of your insightful privacy review feedback (thanks again!) I think this PR should fix:

Remaining work:

This lays the foundation for @pes10k to contribute the proposed cross-site covert channel attack discussed in #197 (or the WG can integrate on your behalf with credits, possibly in another PR).


Preview | Diff

- Add a section for Types of privacy and security threats
- Add a section for Mitigation strategies
@anssiko anssiko requested a review from kenchris June 2, 2023 09:21
@anssiko
Copy link
Member Author

anssiko commented Jun 2, 2023

Since PR Preview is having a bad day, here's a manually created preview:

Preview

You can compare it with the current text.

@kenchris
Copy link
Contributor

kenchris commented Jun 2, 2023

Thanks you @anssiko

@anssiko anssiko requested a review from pes10k June 2, 2023 09:38
@pes10k
Copy link

pes10k commented Jun 2, 2023

I think this is a good change and reorg. I'll leave comments for the mitigations you mentioned and incorporated into the normative parts of the spec in other issues

@anssiko anssiko merged commit 823a43b into main Jun 5, 2023
@anssiko
Copy link
Member Author

anssiko commented Jun 5, 2023

Thanks for your review @pes10k! I made a minor typo fix and merged this PR that addresses #204 and #205.

The following privacy issues remain open and welcome your contributions:

Let's continue discussion in those two issues. Thanks for your contributions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants