Skip to content

Security Considerations: Writing Introduction #272

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

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

simoneonofri
Copy link
Contributor

@simoneonofri simoneonofri commented Jun 12, 2025

Following @johannhof's excellent introduction to privacy, here is the section on security considerations, with various points raised by @Sh-Amir and @ZAnsaroudi.

In particular:

  • Use scenarios
  • External Dependencies
  • Assumptions

Lists of the threats, in progress

@simoneonofri simoneonofri requested a review from a team as a code owner June 12, 2025 18:50
@simoneonofri simoneonofri marked this pull request as draft June 12, 2025 18:50
@timcappalli
Copy link
Collaborator

@simoneonofri this is marked as both draft and awaiting review. Should we start reviewing?

Copy link
Collaborator

@marcoscaceres marcoscaceres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not loving any of this, TBH, as it doesn't provide any actual considerations.

The only part I would keep is the "this is part of a larger ecosystem".

We should list the actual considerations we have takes or are adding as we go... like, only available in secure contexts, not available to third-party iframes without a permission policy, the structures must be JSON serializable, calling the methods requires transient activation, etc.

@simoneonofri
Copy link
Contributor Author

simoneonofri commented Jun 16, 2025

@marcoscaceres, thanks for the feedback. This is just the introduction section. Threats/mitigations are arriving.

We considered putting everything in one PR, but we thought doing smaller, more focused PRs would be more practical. Please let me know if you think there are more practical ways to talk about it.

@simoneonofri simoneonofri marked this pull request as ready for review June 16, 2025 01:29
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
@simoneonofri simoneonofri added the security-tracker Group bringing to attention of security, or tracked by the security Group but not needing response. label Jun 23, 2025
link removed
@marcoscaceres
Copy link
Collaborator

We considered putting everything in one PR, but we thought doing smaller, more focused PRs would be more practical. Please let me know if you think there are more practical ways to talk about it.

Yes please. I think this was a good start, but it showed that the approach was overly broad.

Let's please do focused PRs instead.

simoneonofri and others added 7 commits June 25, 2025 22:35
Co-authored-by: Tim Cappalli <tim@cappalli.me>
Co-authored-by: Tim Cappalli <tim@cappalli.me>
Co-authored-by: Tim Cappalli <tim@cappalli.me>
Co-authored-by: Tim Cappalli <tim@cappalli.me>
Co-authored-by: Tim Cappalli <tim@cappalli.me>
Co-authored-by: Tim Cappalli <tim@cappalli.me>
removing duplicate step
use Platform to be coherent with Tim's diagrams
@marcoscaceres
Copy link
Collaborator

@simoneonofri ok to close this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
security-considerations security-tracker Group bringing to attention of security, or tracked by the security Group but not needing response.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants