Default SSKeychainQuerySynchronizationMode to Any #55

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

SSKeychain class methods do not work with synced passwords.

Create a keychain item with a synced password. Now invoke SSKeychain
class methods like deletePasswordForService:account: or
accountsForService:. They will fail because the SSKeychainQuery
object they initialize internally uses the
SSKeychainQuerySynchronizationModeNo default.

Defaulting to SSKeychainQuerySynchronizationModeAny ensures the
queries created by SSKeychain class methods will still match as
expected.

Default SSKeychainQuerySynchronizationMode to Any
`SSKeychain` class methods do not work with synced passwords.

Create a keychain item with a synced password. Now invoke `SSKeychain`
class methods like `deletePasswordForService:account:` or
`accountsForService:`. They will fail because the `SSKeychainQuery`
object they initialize internally uses the
`SSKeychainQuerySynchronizationModeNo` default.

Defaulting to `SSKeychainQuerySynchronizationModeAny` ensures the
queries created by `SSKeychain` class methods will still match as
expected.
Owner

soffes commented Apr 2, 2014

Can you merge master into your branch so this can merge cleanly? Thanks!

@soffes soffes closed this in 83e1d8f Apr 2, 2014

wesgibbs commented Apr 2, 2014

Looks like you took care of it 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment