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

Added macos support #151

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bananaRanger
Copy link

Hi @evgenyneu!
thank you for keychain-swift. It saves my time!

But I want to share with you some behavior for macOS.

There was a problem:

I am using keychain in my macOS app to store jwt tokens (access and refresh)(kSecClassGenericPassword) and I have figured out that I can't manage existed data via the same app with another name. I have built two equal applications with different app names:

for example App1.app and App2.app
Note: it's the same build, but with only a different - app file name.

And if I create the keychain item using the first app (App1.app) I can't remove it from the second one.

If I double click on the keychain item in Keychain Access default App I can see that there is only one app in Access Control tab (with the name App1.app).

The Solution

Finally, I have found the solution in this article: Making macOS Keychain behave as iOS keychain

And have made a pull request based on this article.

@aehlke
Copy link

aehlke commented Jun 23, 2022

this is unnecessary if kSecAttrSynchronizable is used right?

@bananaRanger
Copy link
Author

@aehlke absolutely, it needs for the case when you don't want use iCloud sync for macOS

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

Successfully merging this pull request may close these issues.

2 participants