Closed Bug 1375512 Opened 4 years ago Closed 4 years ago

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

Categories

(Core :: DOM: Device Interfaces, enhancement)

Product:
Core
Component:
DOM: Device Interfaces
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla56
Tracking Flags:
Tracking Status
firefox56 --- fixed

People

(Reporter: ttaubert, Assigned: ttaubert)

References

Details

(Whiteboard: [webauthn])

Attachments

(1 file)

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: 4 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.