-
Notifications
You must be signed in to change notification settings - Fork 13
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
add: additional key path for macos #28
base: main
Are you sure you want to change the base?
Conversation
Thanks for your contribution, I had some minor remarks. Would be great to address them, then I can merge and cut a release. |
On MacOS it's normal[1] to add custom certificates to `/Library/Keychains/System.keychain` in addition to `/System/Library/Keychains/SystemRootCertificates.keychain`. This PR now checks both locations and concatenates them [1] https://apple.stackexchange.com/questions/53579/how-is-the-system-keychain-secured-in-os-x
202997c
to
053648f
Compare
I took the liberty to rebase to main and update the code as reviewed. |
The remaining question I have is: from your linked stackoverflow article, "The System Keychain, /Library/Keychains/System.keychain, is a special Keychain for Apple and daemons to use. You should generally avoid using it for user level scripts." So, are you sure that path should be added by default to the root certificates / trust anchors? Is there some authoritative documentation (from Apple) suggesting this? I'm not sure I follow your changes, and think that #20 is related -- so anyone with a macOS machine and some binding experience, it would be great to use the macOS API instead of |
I dug around and couldn't find anything on what to do with ssl certificates specifically, it looks like there is a per-user keychain though, so we could try using that one w/the apple API. But in general all of the programs I've used that require self signed certs (i.e proxies) usually ask users to use the system keychain |
Thanks for your research. When you say "all of the programs I've used that require self signed certs (i.e proxies) usually ask users to use the system keychain", are those being installed into |
On MacOS it's normal[1] to add custom certificates to
/Library/Keychains/System.keychain
in addition to/System/Library/Keychains/SystemRootCertificates.keychain
. This PR now checks both locations and concatenates them[1] https://apple.stackexchange.com/questions/53579/how-is-the-system-keychain-secured-in-os-x