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

establish_connection verbose message inaccurate. #64

Closed
plucena24 opened this issue Mar 8, 2015 · 2 comments
Closed

establish_connection verbose message inaccurate. #64

plucena24 opened this issue Mar 8, 2015 · 2 comments

Comments

@plucena24
Copy link
Contributor

establish_connection method in BaseSSHConnection prints "SSH connection established to {0}:{1}" even if the device the session is attempting to connect to is not a valid host on the network. Perhaps this message should be moved down right before line 96:

self.remote_conn = self.remote_conn_pre.invoke_shell()

@ktbyers
Copy link
Owner

ktbyers commented Mar 10, 2015

Just send a pull request on it :-)

Kirk

On Sun, Mar 8, 2015 at 4:57 PM, plucena24 notifications@github.com wrote:

establish_connection method in BaseSSHConnection prints "SSH connection
established to {0}:{1}" even if the device the session is attempting to
connect to is not a valid host on the network. Perhaps this message should
be moved down right before line 96:

self.remote_conn = self.remote_conn_pre.invoke_shell()


Reply to this email directly or view it on GitHub
#64.

Kirk Byers
CCIE #6243 emeritus
ktbyers@twb-tech.com
http://pynet.twb-tech.com http://pynet.twb-tech.com/blog
(415) 264-6270
Simplify your network management through automation

@plucena24
Copy link
Contributor Author

lol will do =)

@ktbyers ktbyers closed this as completed Mar 14, 2015
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

No branches or pull requests

2 participants