Skip to content

feat(ssh): match crojtini host alias by LAN IP too - #11

Merged
inkatze merged 1 commit into
mainfrom
feat/ssh-crojtini-lan-ip
Apr 29, 2026
Merged

feat(ssh): match crojtini host alias by LAN IP too#11
inkatze merged 1 commit into
mainfrom
feat/ssh-crojtini-lan-ip

Conversation

@inkatze

@inkatze inkatze commented Apr 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • Apply the crojtini SSH config (User, ForwardAgent, SendEnv) when connecting via either the crojtini alias or its LAN IP 192.168.1.100.
  • Keep the Hostname crojtini.local resolution scoped to the crojtini alias only, so direct IP connections still work without a redundant DNS lookup.

Test plan

  • ssh crojtini still resolves via .local and forwards the agent / sends ANTHROPIC_API_KEY.
  • ssh 192.168.1.100 connects as inkatze with agent forwarding and SendEnv applied.
  • ssh -G crojtini and ssh -G 192.168.1.100 show the expected user, forwardagent, sendenv values.

Connecting via 192.168.1.100 now picks up ForwardAgent and SendEnv,
so agent forwarding works whether you ssh by alias or by IP (e.g. when
mDNS isn't resolving over VPN).

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the SSH client configuration so the crojtini-specific SSH settings apply when connecting via either the crojtini alias or its LAN IP, while keeping .local hostname resolution only for the alias.

Changes:

  • Extends the crojtini host stanza to also match 192.168.1.100 (User, ForwardAgent, SendEnv).
  • Adds a follow-up Host crojtini stanza that sets Hostname crojtini.local only when using the alias.

馃挕 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@inkatze
inkatze marked this pull request as ready for review April 29, 2026 20:31
@inkatze
inkatze merged commit b36d5a1 into main Apr 29, 2026
12 of 14 checks passed
@inkatze
inkatze deleted the feat/ssh-crojtini-lan-ip branch April 29, 2026 20:31
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