Skip to content

New Mac keyring entries don't trust all Python - #2247

Merged
rolandwalker merged 1 commit into
mainfrom
RW/macos-keyring-trust-python
Sep 15, 2026
Merged

rolandwalker merged 1 commit into
mainfrom
RW/macos-keyring-trust-python

Conversation

@rolandwalker

Copy link
Copy Markdown
Contributor

Description

Previously, Mac keyring/keychain entries would trust Python, meaning that any Python app could read the credential. With this change, that only happens if the user follows up with "always allow" to the keychain dialog.

This still is not perfect! But apparently the only way to do better on Mac would be to have a mycli shim binary.

Based on dbcli/pgcli#1633 .

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

@rolandwalker rolandwalker self-assigned this Sep 15, 2026
@rolandwalker
rolandwalker force-pushed the RW/macos-keyring-trust-python branch from c1b3a69 to 510f0aa Compare September 15, 2026 15:30
Previously, Mac keyring/keychain entries would trust Python, meaning
that any Python app could read the credential.  With this change, that
only happens if the user follows up with "always allow" to the keychain
dialog.

This still is not perfect!  But apparently the only way to do better on
Mac would be to have a mycli shim binary.

Based on dbcli/pgcli#1633 .
@rolandwalker
rolandwalker force-pushed the RW/macos-keyring-trust-python branch from 510f0aa to ce806af Compare September 15, 2026 15:34
@rolandwalker
rolandwalker merged commit 03d4b5b into main Sep 15, 2026
12 checks passed
@rolandwalker
rolandwalker deleted the RW/macos-keyring-trust-python branch September 15, 2026 15:38
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.

1 participant