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

Allowed empty ssh password for remote launching #1826

Merged
merged 1 commit into from
Feb 7, 2020
Merged

Allowed empty ssh password for remote launching #1826

merged 1 commit into from
Feb 7, 2020

Conversation

5tan
Copy link
Contributor

@5tan 5tan commented Oct 6, 2019

Bug in roslaunch doesn't allow to use empty ssh passwords.

E.g. If we try following machine definition:

<machine
  name="mmm"
  address="10.20.73.2"
  user="root"
  password=""
  timeout="10"
  env-loader="/path/to/remote-env.sh"
/>

Empty string is going to be replaced with None here:

self.password = password or None

(because empty strings are being treated as False in boolean operations).
Then, instead of authenticating using empty password, roslauch will attempt to use a key:
if not password: #use SSH agent

After the change (this pull request) everything works as expected:

Case Result
password="abc" authentication using password
password="" authentication using (empty) password
password attribute absent in machine definition authentication using key

@dirk-thomas
Copy link
Member

Thanks for the patch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants