search

mattrajca / sudo-touchid

A fork of `sudo` with Touch ID support.
1.54k stars 47 forks source link

Not working in 10.12.1 #5

Closed j-mcnally closed 7 years ago

j-mcnally commented 7 years ago

Hey there. I have a later 2016 mac book pro.

Love the project but I just get a password prompt when I run sudo.

Wanted to make sure I am not doing something wrong.

I know you said it might not work in 10.12.

Anything I can do to help?

j-mcnally commented 7 years ago

Ohhhh hmmm. the code i cloned looks like it didnt have biometrics enabled.

Odd.

j-mcnally commented 7 years ago

hmm i fixed and recompiled but now im not getting any prompts.

j-mcnally commented 7 years ago

More updates. I am seeing 

LAClient: __52-[LAClient evaluatePolicy:options:uiDelegate:reply:]_block_invoke -> (null), Error Domain=com.apple.LocalAuthentication Code=-7 "No fingers are enrolled with Touch ID." UserInfo={NSLocalizedDescription=No fingers are enrolled with Touch ID.} on <LAClient: 0x7ff302000180>

but i have 3 fingers

j-mcnally commented 7 years ago

I think the issue is its looking for root's fingers....

j-mcnally commented 7 years ago

yep that was exactly it. i enabled the root user and added a finger print and it works. any way to make this use the local user's finger print?

mattrajca commented 7 years ago

Hi @j-mcnally,

I currently don't have a MacBook with Touch ID on hand. What happens if you pass: 0x3f0 for the policy in evaluatePolicy:? It should use the current user's fingerprints instead of root's.

mattrajca commented 7 years ago

This should be fixed in ToT.

j-mcnally commented 7 years ago

Sorry for not getting back to you. Will try thanks.