Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ssh_config.GetCanonicalCase to print correct key case #10

Merged
merged 2 commits into from
Feb 11, 2022

Conversation

StephenBrown2
Copy link

Adds canonical casing to print output, using the GetCanonicalCase func added in PRs to ssh_config

Example:

Host ssh.github.com gist.github.com github.com github
    Hostname ssh.github.com
    IdentitiesOnly yes
    IdentityFile ~/.ssh/id_ed25519
    User git

gets displayed appropriately as:

	gist.github.com(~/.ssh/config) -> [email protected]:22
	    IdentitiesOnly = yes
	    IdentityFile = ~/.ssh/id_ed25519

	github(~/.ssh/config) -> [email protected]:22
	    IdentitiesOnly = yes
	    IdentityFile = ~/.ssh/id_ed25519

	github.com(~/.ssh/config) -> [email protected]:22
	    IdentitiesOnly = yes
	    IdentityFile = ~/.ssh/id_ed25519

	github.me(~/.ssh/config) -> [email protected]:22
	    IdentitiesOnly = yes
	    IdentityFile = ~/.ssh/id_ed25519

	ssh.github.com(~/.ssh/config) -> [email protected]:22
	    IdentitiesOnly = yes
	    IdentityFile = ~/.ssh/id_ed25519

TODO: Remove reference to github.com/StephenBrown2/ssh_config when kevinburke/ssh_config#39 or xwjdsh/ssh_config#1 get merged

TODO: Remove reference to github.com/StephenBrown2/ssh_config when kevinburke/ssh_config#39 or xwjdsh/ssh_config#1 get merged
@xwjdsh xwjdsh force-pushed the get-canonical-case branch from 8815e54 to 2711e35 Compare February 11, 2022 05:54
@xwjdsh xwjdsh force-pushed the get-canonical-case branch from 2711e35 to 5b860c5 Compare February 11, 2022 06:14
@xwjdsh xwjdsh merged commit cfb8a4b into xwjdsh:master Feb 11, 2022
@xwjdsh
Copy link
Owner

xwjdsh commented Feb 11, 2022

Thank you for your contribution.

@xwjdsh
Copy link
Owner

xwjdsh commented Feb 11, 2022

Included in v0.5.3.

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.

3 participants