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

Update handler.py #144

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update handler.py #144

wants to merge 1 commit into from

Conversation

meramsey
Copy link

@meramsey meramsey commented Apr 6, 2020

Add SSH Agent key support.

Allows for SSH Agent use and key support if the local OS is setup for it thanks to Paramikos support of this.

Courtesy of this excerpt
https://github.com/paramiko/paramiko/blob/ae3d0febef17a8ece5268bbf6c210a30573ce800/demos/demo.py#L41-L59

and my mod adding "allow_agent=True, look_for_keys=True," to the string
ssh.connect(*args, allow_agent=True, look_for_keys=True, timeout=options.timeout)

This is working perfectly for me on Linux project and should work in Windows as well.
http://docs.paramiko.org/en/stable/api/agent.html

This will allow for easy switching of servers and key management for those embedding into custom apps.

Add SSH Agent key support.

Allows for SSH Agent use and key support if the local OS is setup for it thanks to Paramikos support of this.

Courtesy of this excerpt
https://github.com/paramiko/paramiko/blob/ae3d0febef17a8ece5268bbf6c210a30573ce800/demos/demo.py#L41-L59

and my mod adding "allow_agent=True, look_for_keys=True," to the string
ssh.connect(*args, allow_agent=True, look_for_keys=True, timeout=options.timeout)

This is working perfectly for me on Linux project and should work in Windows as well.
http://docs.paramiko.org/en/stable/api/agent.html

This will allow for easy switching of servers and key management for those embedding into custom apps.
@huashengdun
Copy link
Owner

Do you think it is really a good idea to enable these stuffs(allow_agent, look_for_keys) on your web server ?

@meramsey
Copy link
Author

meramsey commented Apr 7, 2020 via email

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