log-cache-release icon indicating copy to clipboard operation
log-cache-release copied to clipboard

Used Prometheus package is pinned to a really old version

Open chombium opened this issue 1 year ago • 2 comments

At SAP are running security checks on the Log Cache's codebase and we (me and @zpascal) have seen that the used Prometheus package is pinned to a really old version. The pinned version is 2.13 from October 2019 and the current version is 2.44. I've went through the code today and I've seen that the only place where the Prometheus package is used is in promql.go.

Why is the Prometheus package pinned to an old version? What kind of incompatibilities are there with the newer versions?

As the the Prometheus package is used only in one file I guess it won't be hard to refactor it, so that the newest stable version can be used...

chombium avatar Jun 16 '23 08:06 chombium

Hi @chombium, unfortunately there were many breaking changes in the prometheus package between October 2019 and now, which makes this a non-trivial change. That package is not intended for stable, public consumption, and pulling it in was a questionable choice from a maintenance perspective IMO. Even if we fix the code now, we should expect breaking changes in the future that will make this process hard once again. Check out Prometheus' own description of the go code versioning: https://github.com/prometheus/prometheus#prometheus-code-base.

I did attempt to fix this up some time ago, but never saw it through. The changes I had made are on this branch: https://github.com/cloudfoundry/log-cache-release/tree/fix/prometheus-dependency.

I'm in favor of fixing this, not sure when I would get to it though. PRs would be welcome 😄

ctlong avatar Jun 16 '23 15:06 ctlong

Hi @ctlong, I've taken a quick look at your branch and saw that there are too many things going on a part from the prometheus changes. I suggest that we create a new PR based on the current state in the main branch and your initial changes. We will prepare a PR.

chombium avatar Jun 21 '23 08:06 chombium