Skip to content

Include port and socket in keyring identifier#1513

Merged
rolandwalker merged 1 commit intomainfrom
RW/include-port-and-socket-with-keyring-identifier
Feb 7, 2026
Merged

Include port and socket in keyring identifier#1513
rolandwalker merged 1 commit intomainfrom
RW/include-port-and-socket-with-keyring-identifier

Conversation

@rolandwalker
Copy link
Contributor

Description

It is possible to have different credentials when connecting to the same host on different ports, for instance.

Incidentally recast a variable name as keychain_identifier.

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 Feb 6, 2026
It is possible to have different credentials when connecting to the
same host on different ports, for instance.

Incidentally recast a variable name as "keychain_identifier".
@rolandwalker rolandwalker force-pushed the RW/include-port-and-socket-with-keyring-identifier branch from 9cee7ed to b4bc564 Compare February 6, 2026 10:15
@rolandwalker rolandwalker changed the title Include port and socket with keyring identifier Include port and socket in keyring identifier Feb 6, 2026
@rolandwalker rolandwalker merged commit e908954 into main Feb 7, 2026
8 checks passed
@rolandwalker rolandwalker deleted the RW/include-port-and-socket-with-keyring-identifier branch February 7, 2026 08:36
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