passforios icon indicating copy to clipboard operation
passforios copied to clipboard

iOS 12 - Password AutoFill Keyboard Suggestion Enhancement

Open mmcdole opened this issue 6 years ago • 11 comments

I'm currently using the latest 0.5.1 (15) beta which has the new iOS 12 Password Autofill extension support. This is covered by issue #196.

I'd like to suggest an enhancement to this functionality as it currently is implemented in beta.

When I have 1Password's Autofill functionality enabled in Settings->Passwords Accounts->AutoFill Passwords, I'll get password account suggestions directly on the keyboard bar itself that I can tap for a matching app/website.

It looks like this: password1

Notice the password suggestion that it pulled from the AutoFill provider and showed directly in the keyboard. I don't have to tap the "key icon" and then select the entry from the list.

If I do the same while having passforios I get the following with no suggestions: password2

I'd love to see the same password suggestions when using fields in apps and websites!

mmcdole avatar Nov 15 '18 22:11 mmcdole

It has been in my todo list to a while. But I haven't got time to implement this. T_T Pull requests are always welcome!

yishilin14 avatar Nov 27 '18 11:11 yishilin14

Should Pass for iOS show up as an app in the Settings > "Passwords and Accounts" > Autofill Passwords page on iOS 12?

jmpolom avatar Dec 20 '18 12:12 jmpolom

@jmpolom I implemented that in September. @mssun Would you please release a new version asap?

yishilin14 avatar Dec 20 '18 12:12 yishilin14

Any news on this?

jbddc avatar Jan 14 '19 11:01 jbddc

A new version has been submitted, and it's still under review.

mssun avatar Jan 14 '19 17:01 mssun

@jbddc A new version (0.5.1) has been published in App Store.

mssun avatar Jan 14 '19 20:01 mssun

Cheers for that @mssun! 🙌

jbddc avatar Jan 14 '19 22:01 jbddc

@yishilin14 I've tried building the latest version from master and account suggestions aren't visible so perhaps this isn't there at this stage? Password autofill works fine in general, it's just that the account usernames aren't displayed on the QuickType bar.

davidjb avatar Feb 27 '19 13:02 davidjb

Showing username on the QuickType bar has not been implemented.

As far as I can recall, in order to support this feature, Pass should maintain a separated list of passwords (at least domain names and usernames), which I don't think is very easy to implement. Also, I am not sure whether we should decrypted all usernames and save the decrypted list.

yishilin14 avatar Feb 27 '19 13:02 yishilin14

Yep, @yishilin14 seems to be via ASCredentialIdentityStore. Apple indicates that the list of identities is stored on the device only and only available to AutoFill (https://developer.apple.com/videos/play/wwdc2018/721/) but yes, I agree if it's going to happen, opt-in would be good.

Mozilla's got an example implementation in its lockbox-ios project (https://github.com/mozilla-lockbox/lockbox-ios/blob/473dcd393fd97b3ad10f7299a5fa534fa13f83cd/Shared/Store/CredentialProviderStore.swift#L86)

davidjb avatar Mar 03 '19 23:03 davidjb

There is no need to maintain separate list of domain names or usernames. This can be implemented in a very simple way.

Imagine there are 3 items in keychain: github.com, github.com (alternate), google.com.

  1. We can extract domain names as a first word here — if it matches domain regex of course.
  2. Username would be second word in round braces. If it doesn't exist it can be empty or 'default' etc. So for 2nd case: alternate.

paulmillr avatar Apr 12 '20 05:04 paulmillr