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

Implement an SSH config parser #181

Merged
merged 5 commits into from
Apr 4, 2024
Merged

Implement an SSH config parser #181

merged 5 commits into from
Apr 4, 2024

Conversation

kke
Copy link
Contributor

@kke kke commented Mar 13, 2024

There are several problems in both of the available ssh config parsers for go. After looking at the spec, it's no wonder, it's quite complicated.

For specifics, see the README.

@kke kke added the enhancement New feature or request label Mar 13, 2024
@kke kke force-pushed the ssh-config-parser branch 2 times, most recently from b2de86f to d8a907e Compare March 20, 2024 08:10
protocol/ssh/sshconfig/set.go Fixed Show fixed Hide fixed
protocol/ssh/sshconfig/set.go Fixed Show fixed Hide fixed
@kke kke force-pushed the ssh-config-parser branch from 7ced867 to 4f33afb Compare April 3, 2024 11:23
sshconfig/set.go Fixed Show fixed Hide fixed
sshconfig/set.go Fixed Show fixed Hide fixed
@kke kke force-pushed the ssh-config-parser branch 6 times, most recently from 7bac8a7 to b24c5f7 Compare April 4, 2024 07:33
kke added 4 commits April 4, 2024 10:56
Signed-off-by: Kimmo Lehto <[email protected]>
Signed-off-by: Kimmo Lehto <[email protected]>
Signed-off-by: Kimmo Lehto <[email protected]>
@kke kke force-pushed the ssh-config-parser branch from b24c5f7 to fd7d2ef Compare April 4, 2024 07:56
@kke kke changed the title (wip) Implement an SSH config parser Implement an SSH config parser Apr 4, 2024
@kke kke marked this pull request as ready for review April 4, 2024 09:42
@kke kke merged commit e63043c into main Apr 4, 2024
10 checks passed
@kke kke deleted the ssh-config-parser branch April 4, 2024 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant