Skip to content

fix: remove HPC usernames and SetEnv TERM from ssh configs (regression from PR #39)#50

Merged
stanfish06 merged 1 commit into
masterfrom
claude/nice-cori-ldAd9
May 11, 2026
Merged

fix: remove HPC usernames and SetEnv TERM from ssh configs (regression from PR #39)#50
stanfish06 merged 1 commit into
masterfrom
claude/nice-cori-ldAd9

Conversation

@stanfish06
Copy link
Copy Markdown
Owner

Closes #49

Problem

PR #39 claimed to fix ssh/config and hpc/ssh/config but the changes were never applied. Both files still had live HPC usernames (zyyu, zyu14) and ssh/config's Host * still had SetEnv TERM=xterm-256color.

Changes

ssh/config

  • User zyyu# User <username> for greatlakes, greatlakes1greatlakes5
  • User zyu14# User <username> for bridges2
  • Removed SetEnv TERM=xterm-256color from Host *

hpc/ssh/config

  • User zyyu# User <username> for greatlakes

Test plan

  • grep -r 'zyyu\|zyu14' ssh/ hpc/ssh/ — no matches
  • grep 'SetEnv' ssh/config — no matches
  • SSH to greatlakes works after adding User back locally

Generated by Claude Code

…n from PR #39)

PR #39 claimed to redact `zyyu`/`zyu14` User directives and remove
SetEnv TERM=xterm-256color from ssh/config and hpc/ssh/config, but those
changes were never applied. This commit completes the original fix:

- ssh/config: replace User zyyu/zyu14 with # User <username> placeholder
  for all greatlakes* and bridges2 hosts; remove SetEnv TERM=xterm-256color
  from Host * catch-all
- hpc/ssh/config: replace User zyyu with # User <username>

https://claude.ai/code/session_0177hK1PcXPCCaGbRduJ3dNA
@stanfish06 stanfish06 merged commit 57482bc into master May 11, 2026
1 check passed
stanfish06 added a commit that referenced this pull request May 12, 2026
…54)

Commit af60502 re-added this line on the same branch as PR #50, undoing
the intentional removal in 57482bc. SetEnv TERM overrides the tmux-256color
terminal on remote sessions and is silently ignored by most HPC clusters
that do not set AcceptEnv TERM in sshd_config.
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.

info leak: HPC usernames (zyyu, zyu14) and SetEnv TERM still present in ssh configs — PR #39 regression

2 participants