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

Address privacy issues in P&S explainer #780

Merged
merged 5 commits into from
Aug 15, 2019

Conversation

johnpallett
Copy link
Contributor

Hello! This update to the privacy & security explainer adds more detail about various threat vectors, and explains the principles behind fingerprinting and profiling. It also addresses privacy browsing modes.

Specifically the PR resolves #748 and resolves #750

@NellWaliczek eager to get your feedback!

Copy link
Member

@toji toji left a comment

Choose a reason for hiding this comment

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

Some formatting and phrasing changes suggested. I'll rely on @NellWaliczek to better evaluate the privacy policies.

Copy link
Member

@NellWaliczek NellWaliczek left a comment

Choose a reason for hiding this comment

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

Looks pretty good! Just a few nits

Specific approaches to mitigating device fingerprinting are up to the user agent who is best equipped to evaluate the actual threat on a given platform using the platform's APIs.

### User Profiling
This explainer prioritizes highly the protection of sensitive user characteristics. If there is a reasonable possibility that a reliable signal for a sensitive characteristic exists for some population of users, then [explicit consent](#explicit-consent) is strongly recommended. For example, [explicit consent](#explicit-consent) is strongly recommended before exposing data that sites might use to reliably infer sensitive user characteristics such as race, gender, or age.
Copy link
Member

Choose a reason for hiding this comment

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

"sensitive characteristics" could probably use a stronger, concrete definition. Perhaps in the same initial paragraph regarding "sensitive data"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've updated this to refer to 'demographic' characteristics instead. WDYT?

Copy link
Contributor Author

@johnpallett johnpallett left a comment

Choose a reason for hiding this comment

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

Thank you! Updated text per suggestions; one open question about how to describe user profiling data (I've suggested 'demographic characteristics', WDYT?)

Specific approaches to mitigating device fingerprinting are up to the user agent who is best equipped to evaluate the actual threat on a given platform using the platform's APIs.

### User Profiling
This explainer prioritizes highly the protection of sensitive user characteristics. If there is a reasonable possibility that a reliable signal for a sensitive characteristic exists for some population of users, then [explicit consent](#explicit-consent) is strongly recommended. For example, [explicit consent](#explicit-consent) is strongly recommended before exposing data that sites might use to reliably infer sensitive user characteristics such as race, gender, or age.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've updated this to refer to 'demographic' characteristics instead. WDYT?

@toji
Copy link
Member

toji commented Aug 8, 2019

LGTM, but I'll let @NellWaliczek give the final approval and merge. Thanks!

Copy link
Member

@NellWaliczek NellWaliczek left a comment

Choose a reason for hiding this comment

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

LGTM! I noticed two typos, but other than that, ship it!

@johnpallett
Copy link
Contributor Author

Fixed the typos that Nell caught

@toji toji merged commit acbd884 into immersive-web:master Aug 15, 2019
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.

Address private browsing modes in P&S explainer Privacy & Security explainer: Clarify threat vectors
3 participants