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

gosu carries many CVE and appears unused #22

Closed
jrwren opened this issue May 29, 2024 · 4 comments
Closed

gosu carries many CVE and appears unused #22

jrwren opened this issue May 29, 2024 · 4 comments

Comments

@jrwren
Copy link
Contributor

jrwren commented May 29, 2024

Would you be open to a patch which removes the unused gosu?

@AustinMutschler
Copy link

While gosu is not unused, I agree that it should be removed. They have had a CVE for a few months now because they are compiling with an EOL Go version. When asked to upgrade, the maintainer did but then never released a new build. This is not acceptable in most company security policies. I agree with the maintainer of gosu that it has no impact, I do not agree that there is no reason to release a new version.

Redis used gosu for the "--user" entry flag. Is there a way we can do this without gosu?

@jrwren
Copy link
Contributor Author

jrwren commented Jun 4, 2024

Similar to redis/docker-library-redis#401 (comment)

I believe that ubuntu (edit: sorry, debian) already has runuser OOTB and alpine can apk add runuser to get the same functionality as gosu.

update: never mind. The semantics of runuser are not the same as gosu and rather than exec it does fork and exec.

@tianon
Copy link
Contributor

tianon commented Jun 4, 2024

setpriv is the alternative you're probably looking for.

@roshkhatri
Copy link
Member

Closing this as we have replaces gosu with setpriv

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

No branches or pull requests

4 participants