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

"no side channels" subsection issues #205

Closed
pes10k opened this issue Mar 28, 2023 · 2 comments
Closed

"no side channels" subsection issues #205

pes10k opened this issue Mar 28, 2023 · 2 comments
Labels
privacy-needs-resolution Issue the Privacy Group has raised and looks for a response on.

Comments

@pes10k
Copy link

pes10k commented Mar 28, 2023

This issue is being filed as part of the requested PING review and was broken off from this previous issue

One sub-section is called "no side channels." I found this text confusing since, i. the text in the section seems mostly unrelated to side channels, and ii. "no side channels" is a goal, not a design feature (at least w/o some form of formal modeling thats not present in this spec, and not common in specs).

@anssiko
Copy link
Member

anssiko commented May 19, 2023

@pes10k thanks again. This section was inherited from the earlier version of the API and benefits from a refresh.

I recommend this section to be split into a description of the generic side-channel attack in this context (if deemed relevant for the revised API) and its mitigation, both parts rewritten.

The generic for this split discussed in #204.

@anssiko
Copy link
Member

anssiko commented Jun 5, 2023

This was fixed by #216

The confusing "no side channels" section was removed as part of the Security and privacy considerations reorganization.

@anssiko anssiko closed this as completed Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
privacy-needs-resolution Issue the Privacy Group has raised and looks for a response on.
Projects
None yet
Development

No branches or pull requests

2 participants