Skip to content
This repository has been archived by the owner on Jul 6, 2023. It is now read-only.

heketi-deps release tarball bundles x/net/html with CVE #1372

Closed
nixpanic opened this issue Sep 26, 2018 · 6 comments
Closed

heketi-deps release tarball bundles x/net/html with CVE #1372

nixpanic opened this issue Sep 26, 2018 · 6 comments

Comments

@nixpanic
Copy link
Contributor

Kind of issue

Bug

Observed behavior

Bug 1633022 has been reported against the Heketi package in Fedora. CVE-2018-17142 relates to the fix in golang/go#27702.

It seems Heketi itself does not call html.Parse() itself, nor is x/net/html listed in the glide.lock. However, it still gets included in the heketi-deps tarball. Probably some other package depends on it?

Expected/desired behavior

No known CVEs in the latest release tarballs.

@nixpanic
Copy link
Contributor Author

nixpanic commented Oct 7, 2018

@phlogistonjohn it is much appreciated if an updated heketi-deps tarball can get released!

@phlogistonjohn
Copy link
Contributor

I'm not sure what the best approach for handling this will be. Let's discuss with @obnoxxx @raghavendra-talur .

@sidhax
Copy link

sidhax commented Nov 2, 2018

Does Heketi provide any html interface/api or use x/net/html ?

@phlogistonjohn
Copy link
Contributor

Does Heketi provide any html interface/api or use x/net/html ?

No. It's bundled in because of golang's wonderful approach to dependencies. I hope to do a dependency refresh soon (famous last words). I hope that will allow us to close this issue.

@phlogistonjohn
Copy link
Contributor

(Ahh. closed too many issues I had an itchy trigger finger)

@phlogistonjohn
Copy link
Contributor

A full refresh of dependencies was done for heketi v10.4.0. Soon is a relative term. Closing.

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

No branches or pull requests

3 participants