-
Notifications
You must be signed in to change notification settings - Fork 3
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 remote port forwarding #178
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #178 +/- ##
==========================================
- Coverage 86.54% 86.50% -0.05%
==========================================
Files 9 9
Lines 907 904 -3
==========================================
- Hits 785 782 -3
Misses 122 122
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Hi @superstar54, if this is ready, you can request for review next time. can you see the button on the left? |
@@ -45,6 +45,11 @@ | |||
|
|||
ssh {user}@{hostname} -NfL {port}:localhost:{port} | |||
|
|||
If you visit the remote machine through a jump server, you run (replace the J_IP with the jump server IP): | |||
|
|||
ssh -J {user}@J_IP {user}@{hostname} -NfL {port}:localhost:{port} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is that possible to get the J_IP
if you are already ssh to the remote through SSH, this command is suppose to be copy-paste and run.
@@ -87,7 +92,7 @@ def callback(ctx, param, value): # noqa: U100 | |||
)(cmd) | |||
|
|||
|
|||
@click.group(context_settings={"help_option_names": ["-h", "--help"]}) | |||
@click.group() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you rebase the branch? I guess you force push and override the new changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@superstar54 thanks, just some minor request.
Add a example command for remote port forwarding when user login in through a jump server.