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

Issue #45: Security and Privacy Considerations #104

Merged
merged 5 commits into from Jun 3, 2015

Conversation

mfoltzgoogle
Copy link
Contributor

This PR drafts the Security and Privacy Considerations following discussion in #45. @tidoust and others, please review.

@tidoust
Copy link
Member

tidoust commented Jun 3, 2015

Please find a few comments below. The rest looks good. I would merge the pull request in any case, the text provides a very good basis for that section!

Cross-origin access

  • "the URL that started the presentation": couldn't this mean the URL of the opening context? Change to "the URL of the presentation session", perhaps?
  • I would drop the paragraph starting from "We could further restrict...". We're not going to do that in practice, so I don't see what it brings. Perhaps rephrase the whole paragraph as: "This design allows controlling contexts from different domains to connect to a shared presentation resource. The security of the presentation ID prevents arbitrary pages from connecting to an existing presentation.".
  • I would drop "the charter envisions", or use "the group envisions" instead (most readers won't know what a charter is and that's not a useful concept in a spec)

Temporary identifiers and browser state

I would drop "Again, one possible solution would be to restrict the API to secure contexts" since my understanding is that we would prefer not to do that, and replace with an open issue such as:

<p class="open-issue">
  Should we restrict the API to some extent in non secure contexts?
</p>

@mfoltzgoogle
Copy link
Contributor Author

All good suggestions @tidoust. I have incorporated them into the updated PR and will now merge.

mfoltzgoogle added a commit that referenced this pull request Jun 3, 2015
Issue #45: Security and Privacy Considerations
@mfoltzgoogle mfoltzgoogle merged commit c3928ca into gh-pages Jun 3, 2015
@mfoltzgoogle mfoltzgoogle deleted the security-privacy-issue-45 branch June 3, 2015 22:35
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.

None yet

2 participants