Skip to content

Remove personal SSH hostnames from tracked config; switch to local include - #139

Closed
stanfish06 with Copilot wants to merge 2 commits into
masterfrom
copilot/weekly-repo-review-2026-w30
Closed

stanfish06 with Copilot wants to merge 2 commits into
masterfrom
copilot/weekly-repo-review-2026-w30

Conversation

Copilot AI commented Jul 20, 2026 •

Copy link
Copy Markdown
Contributor

This weekly review identified an information-leak risk in recently changed configs: personal/home-infra hostnames were committed in public ssh/config. This change removes those host-specific entries from tracked config and keeps support for local host definitions via an untracked include path.

  • Scope

    • ssh/config only.
  • Security / privacy

    • Removed personal host blocks (nixos-beelink-1, nixos-gmktec-1) from the repo-managed SSH config.
  • Config structure

    • Added a local include so machine-specific hosts can be defined outside the public repository:
      Include ~/.ssh/local/*.conf
    • Shared/public host definitions (e.g., HPC endpoints) remain in tracked config unchanged.
  • Operational impact

    • Users who relied on removed personal host aliases should define them in ~/.ssh/local/*.conf.

Verified: diff correctly drops only the two personal nixos-* blocks and preserves all HPC entries (greatlakes*, bridges2) and the shared Host * block unchanged. The Include line is placed before any Host blocks, so locally-defined hosts get first-match priority without conflicting with the tracked entries.

Copilot AI linked an issue Jul 20, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Review commits and PRs for week 30 of 2026 Remove personal SSH hostnames from tracked config; switch to local include Jul 20, 2026
Copilot AI requested a review from stanfish06 July 20, 2026 11:46
@stanfish06
stanfish06 marked this pull request as ready for review July 20, 2026 11:56
@stanfish06 stanfish06 closed this Sep 21, 2026
@stanfish06

Copy link
Copy Markdown
Owner

Closing as superseded by #148 — the chezmoi migration (7398c37) moved the live ssh config source to home-chezmoi/private_dot_ssh/config, which had drifted further (6 personal hosts, not the 2 this PR fixed in the now-secondary mirror). #148 fixes both files.


Generated by Claude Code

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.

ssh/config: personal home-server hostnames exposed in public repo Weekly repo review – 2026-W30

2 participants