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

Errors when using fish shell #8

Closed
budric opened this issue Apr 6, 2016 · 7 comments
Closed

Errors when using fish shell #8

budric opened this issue Apr 6, 2016 · 7 comments

Comments

@budric
Copy link

budric commented Apr 6, 2016

Hi,

Great software package! Thank you.

When the remote user is configured to use fish shell, the check fails when executing:
set -o pipefail; rpm -qa --queryformat '%{NAME} %{VERSION} %{RELEASE}'
I believe the issue is tab characters between %{NAME} %{VERSION} %{RELEASE} headers.

It's not an issue with unknown option as debug output suggests.

Some debug output from vuls near the error:

[Apr  6 17:18:41] DEBUG [zabbix:22] SSH executed. cmd: set -o pipefail; rpm -qa --queryformat '%{NAME}  %{VERSION}  %{RELEASE}
', status: <nil>
stdout: 
set: Unknown option '-o'
Standard input: set -o pipefail; rpm -qa --queryformat '%{NAME} %{VERSION}  %{RELEASE}
                ^
hawkey  0.5.8   2.git.0.202b194.el7
<snip> rest of my packages in similar format </snip>

stderr: 

[Apr  6 17:18:41] ERROR [zabbix:22] Failed to scan installed packages

@aidabeorn
Copy link

It does seem to rely on bash on the remote end, the issue is infact set -o pipefail, fish doesn't have -o, zsh(which I use) doesn't have the option pipefail until zsh 5.0.8

@budric
Copy link
Author

budric commented Apr 7, 2016

Yes, you're right it is the set -o pipefail. My mistake.

@kotakanbe
Copy link
Member

I am installing fish-shell (first time in my life)

@kotakanbe
Copy link
Member

fish shell is so cool!!!

@kotakanbe
Copy link
Member

fish-shell is awesome.
I tried but It seems difficult to support fish-shell or old zsh.
Please create a user with bash login shell on target servers.

@kotakanbe
Copy link
Member

I will update README about this.
Thanks for reporting.

@kotakanbe
Copy link
Member

update README.md
ac77cc1

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

3 participants