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

Possible shortcoming in KIALI-SECURITY-001 detection script #3135

Closed
cpick opened this issue Aug 19, 2020 · 3 comments · Fixed by kiali/kiali.io#294
Closed

Possible shortcoming in KIALI-SECURITY-001 detection script #3135

cpick opened this issue Aug 19, 2020 · 3 comments · Fixed by kiali/kiali.io#294
Assignees
Labels
backlog Triaged Issue added to backlog bug Something isn't working

Comments

@cpick
Copy link

cpick commented Aug 19, 2020

Describe the bug

The detection section of the KIALI-SECURITY-001 bulletin includes the following script:

[ ${VERSION_ENTRIES[0]} -le "1" ] && [ ${VERSION_ENTRIES[1]} -le "15" ] && [ ${VERSION_ENTRIES[2]} -le "0" ] && echo "Your Kiali version is vulnerable"

This will print "Your Kiali version is vulnerable" if the version is >= 1.0.0 && <= 1.15.0, but not if the major version is 0.X.Y.

Versions used
Kiali: < v1

To Reproduce
Steps to reproduce the behavior:

  1. Install an old Kiali (eg v0.20)
  2. Run the detection script from the bulletin

Expected behavior
I believe it should print "Your Kiali version is vulnerable" but it does not.
I could be mistaken if the vulnerability was only introduced in v1 and isn't present in v0.

@cpick cpick added the bug Something isn't working label Aug 19, 2020
@lucasponce
Copy link
Contributor

lucasponce commented Aug 20, 2020

Side question, in which scenarios are Kiali v0.20 being used ?
The Kiali 0.x was a preliminary release and we would like to help potential users to bump into a newer one.

@jshaughn
Copy link
Collaborator

Thanks @cpick. I guess we just didn't think anyone would be running V0.x as it is long out of support for Kiali and not associated with any supported version of Istio, But we can update the script.

@jshaughn jshaughn added team/saturn backlog Triaged Issue added to backlog labels Aug 20, 2020
@cpick
Copy link
Author

cpick commented Aug 20, 2020

@lucasponce I don't know of a good reason why it would still be used, but ancient versions of all software find ways to live on :(

I do think it's useful to have more ways to point out that running an old/unsupported version is a bad idea. Having checks like these properly flag them as vulnerable helps make that case. (I don't get the sense that anyone disagrees with this idea, just putting down my thinking.)

israel-hdez added a commit to israel-hdez/kiali.io that referenced this issue Aug 21, 2020
Detection script was not outputting the expected warning text on all
affected versions. This change should fix the detection test.

Fixes kiali/kiali#3135
mtho11 pushed a commit to mtho11/kiali.io that referenced this issue Nov 18, 2020
Detection script was not outputting the expected warning text on all
affected versions. This change should fix the detection test.

Fixes kiali/kiali#3135
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Triaged Issue added to backlog bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants