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

RESOLVED FIXED in Firefox 56

Status

()

Product:
Core
Component:
DOM: Device Interfaces
Type:
enhancement
Status:
RESOLVED FIXED
Opened:
2 years ago
Updated:
2 years ago

People

(Reporter: ttaubert, Assigned: ttaubert)

Tracking

Version:
unspecified
Target:
mozilla56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

(Whiteboard: [webauthn])

Attachments

(1 attachment)

0002-Bug-1375512-Don-t-require-a-U2F-TokenManager-to-expo.patch
14.90 KB, patch
qdot
: review+
Details | Diff | Splinter Review 
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
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.