Skip to content

build(deps): bump snowflake-prometheus-exporter to resolve CVE-2025-46327 in gosnowflake #3528

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

Conversation

rgoltz
Copy link
Contributor

@rgoltz rgoltz commented May 7, 2025

PR Description

This PR aims to get rid of CVE-2025-46327 / GHSA-6jgm-j7h2-2fqg in Alloy, which is regarding a race condition when checking access to Easy Logging configuration file in gosnowflake package. Once you using a SBOM-based scan against Alloy, it's reporting this vulnerability.

Since github.com/snowflakedb/gosnowflake is an indirect dependency of github.com/grafana/snowflake-prometheus-exporter, I went ahead and created grafana/snowflake-prometheus-exporter#28 - This was reviewed, tested and merged by the team of snowflake-prometheus-exporter.

Afterwards I ran:

go get github.com/grafana/snowflake-prometheus-exporter
go mod tidy

This results in an update of snowflake-prometheus-exporter from v0.0.0-20240813124544 to v0.0.0-20250507154309 and also the needed update of gosnowflake from v1.7.2 to v1.14.0. To get rid of the risk of CVE-2025-46327 we need v1.13.0 or higher.

Copy link
Contributor

@wildum wildum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@wildum
Copy link
Contributor

wildum commented May 21, 2025

@rgoltz there are some conflicts, could you please rebase your branch so that we can merge it?

@rgoltz
Copy link
Contributor Author

rgoltz commented May 21, 2025

Hi @wildum - William, I'm currently not "close" to a PC. It's possible that you pull my branch and rebase from your side with an addtional commit to this PR? - In case I should do it, it will take until next week at minimum. Merci.

@wildum
Copy link
Contributor

wildum commented May 21, 2025

Ok no worries I will take it from here!

@wildum
Copy link
Contributor

wildum commented May 21, 2025

mmh somehow your commit contained an update to the SQL exporter which is making the build fail. I did go get github.com/grafana/snowflake-prometheus-exporter on a clean branch and did not get this dependency. I will open a new PR to bring the change and mention you there

@wildum wildum closed this May 21, 2025
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

Successfully merging this pull request may close these issues.

2 participants