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.

Sign up for GitHub

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

Jump to bottom

Passwords for ProxyJump hosts in .ssh/config are not saved #13936

Closed
dirkpetersen opened this issue Dec 4, 2022 · 4 comments · Fixed by #13940
Closed

Passwords for ProxyJump hosts in .ssh/config are not saved #13936

dirkpetersen opened this issue Dec 4, 2022 · 4 comments · Fixed by #13940
Assignees
@dkocher
Labels
sftp SFTP Protocol Implementation
Milestone
8.5.2

Comments

@dirkpetersen
Copy link

dirkpetersen commented Dec 4, 2022 •
edited

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

When I use the ProxyJump feature the password for the jumphost is never saved (Version 8.5.1 / Windows 11). This is especially problematic since Cyberduck seems to need to re-authenticate each time I hit the upload button so if the files I upload are scattered over multiple folders I have to enter my password 4-5 times which seems a bit hacky

Host jumper
    User user1
    Hostname host1.domain.org

Host *
    ProxyJump jumper

Describe the solution you'd like
save password for jumphost if checked when prompted

Describe alternatives you've considered
Considered ssh keys but are not allowed here
@dkocher dkocher added the sftp SFTP Protocol Implementation label Dec 5, 2022
@dkocher dkocher self-assigned this Dec 5, 2022
@dkocher
Copy link
Contributor

dkocher commented Dec 5, 2022

Should have been resolved with 3e28ad3 but apparently we fail to lookup the saved credentials.

@dkocher dkocher changed the title passwords for ProxyJump hosts in .ssh/config are not saved Passwords for ProxyJump hosts in .ssh/config are not saved Dec 5, 2022
@dkocher dkocher linked a pull request Dec 5, 2022 that will close this issue
Fix #13936. #13940
Merged
@dkocher dkocher added this to the 8.5.2 milestone Dec 5, 2022
@dkocher dkocher closed this as completed in 8040e6b Dec 5, 2022
dkocher added a commit that referenced this issue Dec 5, 2022
@dkocher
Merge branch 'bugfix/GH-13936'
ae97492
@dirkpetersen
Copy link
Author

Awesome, thanks a lot !

@dkocher
Copy link
Contributor

dkocher commented Dec 5, 2022

Awesome, thanks a lot !

It would be great if you can verify the fix with build 8.5.2.38764 available through Preferences… → Update → Snapshot Builds.

@dirkpetersen
Copy link
Author

I tested it and it works very well. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dkocher dkocher

Labels
sftp SFTP Protocol Implementation
Projects
None yet
Milestone
8.5.2
Development

Successfully merging a pull request may close this issue.

Fix #13936. iterate-ch/cyberduck
2 participants
@dkocher @dirkpetersen