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

sshnp: add -u option to specify username to use if sshnpd is not sharing it #202

Closed
gkc opened this issue Jun 14, 2023 · 2 comments
Closed
Assignees
Labels
enhancement New feature or request

Comments

@gkc
Copy link
Contributor

gkc commented Jun 14, 2023

Is your feature request related to a problem? Please describe.

from @cconstab

sshnp shoukd have a -u option to specify the username if the sshnpd was not sharing it.
This is a bug/oversight in sshnp right now.

Describe the solution you'd like

  • add an optional -u option in SSHNP.createArgParser with full option name remote-user-name, description something like, "user name to use in the ssh session on the remote host"
  • in SSHNP, change remoteUserName from late final String remoteUsername to String? remoteUserName
  • add optional remoteUserName parameter to SSHNP constructor to allow remoteUserName to be supplied, and supply the parsed
  • in SSHNP.init - do not call fetchRemoteUserName if a remoteUserName has already been supplied

Describe alternatives you've considered

No response

Additional context

Either wait for #201 to be merged to trunk, or make a new branch based on that PR's branch

@VJag
Copy link
Member

VJag commented Jun 14, 2023

@purnimavenkatasubbu Can you work on this ticket? @srieteja can help if you need.

@purnimavenkatasubbu
Copy link
Member

PR merged to trunk. Closing this issue as fixed.

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

No branches or pull requests

4 participants