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

Upgrade Go version 1.23.4 and pkg github.com/moby/sys/user, golang.org/x/sys #154

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vkamlesh
Copy link

No description provided.

@vkamlesh vkamlesh changed the title Upgrade pkg github.com/moby/sys/user and golang.org/x/sys version Upgrade Go version 1.23.4 and pkg github.com/moby/sys/user, golang.org/x/sys Dec 19, 2024
@tianon
Copy link
Owner

tianon commented Dec 19, 2024

See https://github.com/tianon/gosu/blob/master/SECURITY.md#version-updates

(I don't believe there have been any actual functional updates to the code we use that warrant updating)

@Bezuhlyi
Copy link

Bezuhlyi commented Jan 28, 2025

@tianon Maybe releasing a version that consists of two parts (functional version + golang version -> gosu-1.17_1.23.4) could help with the concern that if there are no functional changes, you should not release the new version just because of the newer golang used for the build? Similar to how they version Flink images.

I'd deeply appreciate having the option to download the newer version of gosu and put it on top of the Filink image rather than wasting time on requesting and justifying an exception for the dozens of golang vulnerabilities highlighted by Jfrog Xray scanner. It's a common request, after all, let's have some solution.

The guides like this https://internetworking.dev/mitigating-gosu-security-concerns/ is crazy.

//cc @vkamlesh

@m0t1x
Copy link

m0t1x commented Jan 29, 2025

Maybe, from the functionality point of view, vulnerabilities detected are not a threat, since docker build process is a local process and maybe gosu is not using those functionalities reported by the scanner. Still, using Golang release which is almost 3 years old, and because the whole update process is really not a complex task, I think it is worth while to do this update. Like suggested by @Bezuhlyi , using a docker like tag convention app-ver_os-ver, which can in your case be app-ver_go-ver is pretty good idea.

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.

4 participants