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.

Sign up for GitHub

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

Jump to bottom

Fixing bug where @ symbol in command was not parsed correctly #9

Open

JohnCMoon wants to merge 1 commit into frowand:master from JohnCMoon:master

JohnCMoon commented Aug 30, 2019

Hi Frank,
I learned about this project at ELC this year and think it's great - thanks for sharing!

I found a small bug where commands with @ in them resulted in an error if your host target did not contain an @. For example:

# ./sersh ttyUSB0 'systemctl status openvpn-client@client'
ERROR: [Errno 21] could not open port /dev/: [Errno 21] Is a directory: '/dev/'

This little fix should stop the host string search from going into the command string when parsing.

Cheers,
John


          Fixing bug where @ symbol in command was not parsed correctly

e09543e

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

Labels

None yet