Skip to content
This repository has been archived by the owner on Sep 7, 2023. It is now read-only.

MacKeyChain uses "Legacy" APIs #67

Closed
ghelyar opened this issue Feb 26, 2020 · 2 comments · Fixed by #126
Closed

MacKeyChain uses "Legacy" APIs #67

ghelyar opened this issue Feb 26, 2020 · 2 comments · Fixed by #126
Assignees
Labels
enhancement New feature or request

Comments

@ghelyar
Copy link

ghelyar commented Feb 26, 2020

The APIs that are being used on MacOS are not deprecated, but are marked as "Legacy" and their use is discouraged.

https://developer.apple.com/documentation/security/keychain_services/keychain_items?language=objc#1659107

This may not be an immediate issue, but Apple may deprecate and then remove them in the future, and they don't support more advanced scenarios such as Sharing Access to Keychain Items Among a Collection of Apps.

The P/Invoke also doesn't use utf-8 encoded strings, so may cause issues with non-ascii characters in the service or account names.

@henrik-me
Copy link

@ghelyar : Thanks a lot for reporting. Added to the backlog. We will happily take a PR helping address this.

@henrik-me henrik-me added the enhancement New feature or request label Feb 26, 2020
@bgavrilMS bgavrilMS added the help wanted Extra attention is needed label Mar 6, 2020
@bgavrilMS
Copy link
Member

Note that Git Credential Manager have written the interop code that allows for this:

https://github.com/microsoft/Git-Credential-Manager-Core/tree/master/src/shared/Microsoft.Git.CredentialManager/Interop/MacOS

@bgavrilMS bgavrilMS self-assigned this Feb 8, 2021
@bgavrilMS bgavrilMS removed the help wanted Extra attention is needed label Feb 8, 2021
@bgavrilMS bgavrilMS linked a pull request Feb 8, 2021 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants