linkwarden icon indicating copy to clipboard operation
linkwarden copied to clipboard

Allow running docker images as non-root user

Open tribut opened this issue 1 year ago • 2 comments

This changes permissions inside the docker image in order to be able to run linkwarden without root permissions.

I decided against creating a special-purpose user as proposed in #701, because this would be a breaking change for existing installs (requires changes to ownership of existing bind-mounts). Instead, I've added configuration to docker-compose.yml that new installs will pick up and allows customization by the user.

Fixes #550 Fixes #655 Fixes #712

tribut avatar Sep 29 '24 10:09 tribut

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Sep 29 '24 10:09 CLAassistant

Hey thanks for the contribution! Gonna merge this soon.

daniel31x13 avatar Oct 01 '24 21:10 daniel31x13

Just took a look at this, there are two major issues after running this:

  1. Well, it doesn't run - keeps giving me an error that it can't reach the database: image

  2. The size of the container increased to 5.37GB: image

daniel31x13 avatar Nov 03 '24 01:11 daniel31x13

Getting this working is critical. There's a lot of helpful info over here. https://github.com/linkwarden/linkwarden/issues/799

discotimetraveler avatar Nov 25 '24 21:11 discotimetraveler

Thanks for taking a look, Daniel. Don't have much free time currently, but will keep this on my list.

If anybody else needs this urgently, feel free to take a look yourselves. I am sure I tried this, so my best guess is the problem is related to permissions of mounted volumes.

tribut avatar Nov 25 '24 22:11 tribut

Hi! I'm confused if the problems with preserved formats are still tied to this or rather things discussed here? And are these commits still problematic or could they be merged?

ed0mm avatar Jan 08 '25 19:01 ed0mm

I made an updated version of this PR here, fixes the Docker image size issue: #1483

holocronweaver avatar Nov 19 '25 13:11 holocronweaver