Don't require a U2F*TokenManager to expose IsRegistered()

RESOLVED FIXED in Firefox 56

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: ttaubert, Assigned: ttaubert)

Tracking

unspecified
mozilla56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

(Whiteboard: [webauthn])

Attachments

(1 attachment)

U2FTokenTransport is the actual interface we should be using in the U2FTokenManager, and that doesn't have IsRegistered() either. We should move those calls into Register() and Sign() as they're implementation-dependent. The HID tokens for example will have to keep calling u2f_register() for tokens where isRegistered()=false when signing, due to privacy reasons.
Attachment #8880411 - Flags: review?(kyle) → review+
Pushed by ttaubert@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/168eddb5129e
Don't require a U2F*TokenManager to expose IsRegistered() r=qDot
https://hg.mozilla.org/mozilla-central/rev/168eddb5129e
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.