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

rpID origin relaxation? #338

Closed
mikewest opened this issue Feb 13, 2017 · 3 comments
Closed

rpID origin relaxation? #338

mikewest opened this issue Feb 13, 2017 · 3 comments

Comments

@mikewest
Copy link
Member

@annevk pointed out the algorithm in https://w3c.github.io/webauthn/#makeCredential which makes use of bits and pieces of document.domain that I would dearly love to remove from the platform. :)

It doesn't look like this (or the corresponding bits of getAssertion()) intend to change the document's origin, but it's not clear to me what impact they do have. Is the intent to support sharing auth tokens cross-origin? If so, could you help me understand why the origin model fails to support the use cases y'all have in mind?

Thanks!

@equalsJeffH
Copy link
Contributor

As @balfanz wrote in https://lists.w3.org/Archives/Public/public-webauthn/2016Aug/0045.html:

I'll point out that the webauthn spec is currently strictly enforcing
same-origin (where origin is defined by scheme-host-port) by requiring that
the so-defined origin is included in the client data. An assertion
generated on one origin won't be valid on another origin.

The PSL [eTLD] dependency is there simply as a recommendation on how to scope key
pairs, meaning that two origins within the same public suffix [eTLD+1] may know the
client by the same public key. See my comment on the original github thread
as to why that is:
w3ctag/design-reviews#97 (comment)

@equalsJeffH
Copy link
Contributor

equalsJeffH commented Feb 14, 2017

see also issue #241 and #256 and PR #319 and whatwg/html#2365

@vijaybh
Copy link
Contributor

vijaybh commented Mar 4, 2017

Seems to have been addressed by #319.

@vijaybh vijaybh closed this as completed Mar 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants