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

Consider providing a Docker image that doesn't run as root #25926

Open
remram44 opened this issue Aug 4, 2023 · 3 comments · May be fixed by #20627
Open

Consider providing a Docker image that doesn't run as root #25926

remram44 opened this issue Aug 4, 2023 · 3 comments · May be fixed by #20627
Labels
S-Minor Impairs non-critical functionality or suitable workarounds exist T-Enhancement

Comments

@remram44
Copy link
Contributor

remram44 commented Aug 4, 2023

Your use case

What would you like to do?

Run the Docker image and have it not use root privileged

Why would you like to do it?

Increased security, usage in environments where root in container is not available

How would you like to achieve it?

Use the image nginxinc/nginx-unprivileged for example

Have you considered any alternatives?

I am building my own images at ghcr.io/remram44/matrix-helm/element but this might as well be official. See the Dockerfile.

Additional context

No response

@t3chguy
Copy link
Member

t3chguy commented Aug 4, 2023

Fixed by #20627

@t3chguy t3chguy linked a pull request Oct 4, 2022 that will close this issue
@remram44
Copy link
Contributor Author

remram44 commented Aug 4, 2023

You could keep both images (like nginx does with nginx-unprivileged) to avoid breaking current users

@t3chguy
Copy link
Member

t3chguy commented Aug 4, 2023

Either sounds sane to me, its up to that contributor if they're still around to pick or for someone else if they wish to take over the contribution

@germain-gg germain-gg added S-Minor Impairs non-critical functionality or suitable workarounds exist Z-Docker labels Aug 7, 2023
@Johennes Johennes removed the Z-Docker label Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-Minor Impairs non-critical functionality or suitable workarounds exist T-Enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants