Skip to content

[Snyk] Security upgrade golang from 1.12.4-alpine to 1.25.1-alpine#40

Open
RealTschoegl wants to merge 1 commit intomasterfrom
snyk-fix-1ec7fd59fbf238386bc5893339778f33
Open

[Snyk] Security upgrade golang from 1.12.4-alpine to 1.25.1-alpine#40
RealTschoegl wants to merge 1 commit intomasterfrom
snyk-fix-1ec7fd59fbf238386bc5893339778f33

Conversation

@RealTschoegl
Copy link

snyk-top-banner

Snyk has created this PR to fix 4 vulnerabilities in the dockerfile dependencies of this project.

Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.

Snyk changed the following file(s):

  • hydra/Dockerfile-e2e

We recommend upgrading to golang:1.25.1-alpine, as this image has only 0 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.

Vulnerabilities that will be fixed with an upgrade:

Issue Score
critical severity Out-of-bounds Write
SNYK-ALPINE39-MUSL-458529
  714  
critical severity Out-of-bounds Write
SNYK-ALPINE39-MUSL-458529
  714  
high severity Improper Certificate Validation
SNYK-ALPINE39-OPENSSL-1089232
  614  
high severity Integer Overflow or Wraparound
SNYK-ALPINE39-OPENSSL-1089235
  614  
high severity NULL Pointer Dereference
SNYK-ALPINE39-OPENSSL-588029
  614  

Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic


Learn how to fix vulnerabilities with free interactive lessons:

🦉 NULL Pointer Dereference

@gitstream-cm
Copy link

gitstream-cm bot commented Sep 4, 2025

Please mark whether you used AI to assist coding in this PR

  • AI Assisted
  • Not AI Assisted

@macroscopeapp
Copy link

macroscopeapp bot commented Sep 4, 2025

Upgrade the e2e build base image in hydra/Dockerfile-e2e to golang:1.25.1-alpine from golang:1.12.4-alpine as a Snyk security upgrade

Modify the FROM directive in Dockerfile-e2e, changing the Go toolchain and Alpine base used for building the e2e image.

📍Where to Start

Start with the FROM line in Dockerfile-e2e.


Macroscope summarized 57630d6.

Copy link

@gitstream-cm gitstream-cm bot left a comment

Choose a reason for hiding this comment

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

✨ PR Review

This security upgrade addresses critical vulnerabilities by updating Golang from 1.12.4 to 1.25.1, which is necessary for security but represents a significant version jump that requires careful validation.

1 issues detected:

🐞 Bug - Massive version upgrade without incremental testing could break application functionality

Details: Upgrading from Go 1.12.4 to 1.25.1 represents a massive version jump spanning 13 major versions. This could introduce breaking changes in language features, module system, standard library, build behavior, and runtime compatibility that may cause the application to fail to build or run correctly.
File: hydra/Dockerfile-e2e (1-1)

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀

@@ -1,4 +1,4 @@
FROM golang:1.12.4-alpine
FROM golang:1.25.1-alpine
Copy link

Choose a reason for hiding this comment

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

🐞 Bug - Major Version Jump: Consider testing the upgrade thoroughly in a non-production environment first. Review Go release notes for breaking changes between versions 1.12 and 1.25, and verify that all application code, dependencies, and build scripts remain compatible with the newer Go version.

Suggested change
FROM golang:1.25.1-alpine
FROM golang:1.21-alpine

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