Skip to content

Update handler.py#144

Open
meramsey wants to merge 1 commit intohuashengdun:masterfrom
meramsey:patch-1
Open

Update handler.py#144
meramsey wants to merge 1 commit intohuashengdun:masterfrom
meramsey:patch-1

Conversation

@meramsey
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
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