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

Update publickey-credentials to be "get-only" policy #370

Merged
merged 1 commit into from
Apr 30, 2020

Conversation

equalsJeffH
Copy link
Contributor

Due to WebAuthn issue w3c/webauthn#1336, the WebAuthn's "publickey-credentials" feature policy is renamed to "publickey-credentials-get", allowing only authentication (aka "getting an assertion") in cross-origin iframes. Credential creation is not allowed in cross-origin iframes, although it remains allowed by default (no feature-policy required) in same-origin-with-ancestors iframes.

See also PR w3c/webauthn#1394, and https://w3c.github.io/webauthn/#sctn-feature-policy

Due to WebAuthn issue #1336, the WebAuthn's "publickey-credentials" feature policy is renamed to "publickey-credentials-get", allowing only authentication (aka "getting an assertion") in cross-origin iframes. Credential creation is not allowed in cross-origin iframes, although it remains allowed by default (no feature-policy required) in same-origin-with-ancestors iframes.

See also w3c/webauthn#1394, and https://w3c.github.io/webauthn/#sctn-feature-policy
@clelland clelland merged commit 0ee2566 into w3c:master Apr 30, 2020
blueboxd pushed a commit to blueboxd/chromium-legacy that referenced this pull request May 1, 2020
The WebAuthn spec has removed the ability to allow cross-origin iframes
to perform Web Authentication MakeCredential requests. This CL restricts
it to just GetAssertion, and reflects the renamed Feature Policy.

Feature Policy spec change:
w3c/webappsec-permissions-policy#370

WebAuthn spec change:
w3c/webauthn#1394

Bug: 993007
Change-Id: I9b5ccf05b5e39a5e5920b475111cdf797dcdb5a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120268
Reviewed-by: Martin Kreichgauer <[email protected]>
Reviewed-by: Jeremy Roman <[email protected]>
Reviewed-by: Ian Clelland <[email protected]>
Commit-Queue: Ken Buchanan <[email protected]>
Cr-Commit-Position: refs/heads/master@{#764743}
Elchi3 pushed a commit to mdn/browser-compat-data that referenced this pull request May 6, 2020
Feature Policy directive controlling WebAuth was renamed from
publickey-credentials to publickey-credentials-get.
Sources:
  Feature Policy change:
    w3c/webappsec-permissions-policy#370
  WebAuthn specification change:
    w3c/webauthn#1394
  Chrome already updated the name:
    https://crbug.com/993007#c9
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.

2 participants