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

UVM extension returns all available verification methods instead of the actual method used. #78

Open
gcbenjamin opened this issue Feb 18, 2020 · 1 comment

Comments

@gcbenjamin
Copy link

gcbenjamin commented Feb 18, 2020

I'm trying to limit the authentication to only allow fingerprint through on my device and not pin or pattern. On a phone device at least I don't believe pin/pattern to be secure as parents are always giving out the pin to their children and pattern leaves a smudge on a dirty screen meaning it's easily visible in the correct light (and also given out to children).

I was hoping the UVM extension would be able to help but it returns the same flags (fingerprint, pin, pattern) every time, no matter which verification method I've used. This is blocking me implementing passwordless authentication on my site (credit cards (PCI compliant) are stored so future purchases are simpler). Is that intended or a bug?

@suzylii
Copy link

suzylii commented Feb 18, 2020

Hi,

This is work as intended. On Android P and earlier, we have implemented a custom fingerprint flow and can conclusively say fingerprint was used on when the user uses that flow. However, on Q+, we rely on the framework to authenticate the user in all cases, using whatever method the user chooses. On Android devices today, that is either fingerprint, passcode, or face id. Unfortunately, the framework does not expose the information on which method was used. Therefore, the value that you see is the logical OR of the bits defined in the specification [1].

[1] https://fidoalliance.org/specs/fido-v2.0-id-20180227/fido-registry-v2.0-id-20180227.html#user-verification-methods

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

No branches or pull requests

3 participants
@gcbenjamin @suzylii and others