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

Fix private keys are not found on Linux/Windows #216

Merged
merged 8 commits into from
Nov 17, 2020

Conversation

fernandezcuesta
Copy link
Collaborator

No description provided.

@pahaz pahaz changed the title Fix #161 Fix private keys are not found on Linux Nov 16, 2020
@pahaz
Copy link
Owner

pahaz commented Nov 16, 2020

#126 this may be related to the same code part

@pahaz
Copy link
Owner

pahaz commented Nov 16, 2020

Sorry for my commit :) feel free to delete it. You are the first %)

@fernandezcuesta
Copy link
Collaborator Author

No problem, I'm booking some time later on and fix this issue. Not as easy as deleting the if condition though.

@pahaz
Copy link
Owner

pahaz commented Nov 16, 2020

Probably we also can use ssh_private_key_password for the default ssh key location. It's also may be helpful for case like so:

tunnel = SSHTunnelForwarder(
    SSH_SERVER_ADDRESS,
    ssh_username=SSH_SERVER_USERNAME,
    ssh_private_key_password=SSH_SERVER_PKEY_PASS,
    remote_bind_address=SSH_SERVER_REMOTE_SIDE_ADDRESS,
)

@pahaz
Copy link
Owner

pahaz commented Nov 16, 2020

And also, #145 is also may be fixed. I think we can check the ~/.ssh path first and then check ~/ssh for windows. But I'm not sure 🤔

@pahaz pahaz changed the title Fix private keys are not found on Linux Fix private keys are not found on Linux/Windows Nov 16, 2020
@fernandezcuesta fernandezcuesta force-pushed the fernandezcuesta-patch-161 branch from 0490132 to 10d3a41 Compare November 16, 2020 23:09
@coveralls
Copy link

coveralls commented Nov 16, 2020

Coverage Status

Coverage increased (+1.04%) to 91.64% when pulling 9739703 on fernandezcuesta-patch-161 into cd374d5 on master.

@fernandezcuesta
Copy link
Collaborator Author

fernandezcuesta commented Nov 17, 2020

regarding #145, I agree on setting ~/.ssh as default config folder for windows as well.

@fernandezcuesta fernandezcuesta merged commit 6d636f2 into master Nov 17, 2020
@fernandezcuesta fernandezcuesta deleted the fernandezcuesta-patch-161 branch November 17, 2020 10:39
@fernandezcuesta fernandezcuesta restored the fernandezcuesta-patch-161 branch November 17, 2020 10:41
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