NathanVaughn / webtrees-docker

Up-to-date Docker image for webtrees with all the bells and whistles.
https://hub.docker.com/r/nathanvaughn/webtrees
MIT License
64 stars 15 forks source link

Container restart resets config file permissions #89

Closed rounakdatta closed 2 years ago

rounakdatta commented 2 years ago
Fatal error: Uncaught ErrorException: parse_ini_file(/var/www/webtrees/data/config.ini.php)

I get this error every time the webtrees container gets restarted. This blocks the application entirely. When I look at the file permissions of the config.ini.php file,

---------- 1 www-data www-data 231 Mar 18 15:12 config.ini.php

Currently I fix it everytime by manually running

chmod 700 config.ini.php

Is this a configuration issue on my side, or an idempotency issue in the Dockerfile? I believe we needn't generate/edit the config.ini.php file everytime.

NathanVaughn commented 2 years ago

Um, that's strange. The entrypoint should be resetting the file permissions to 755 on every container start, not completely nuking the permissions. Maybe I need to add that step again to the end, after the file has undergone modifications

NathanVaughn commented 2 years ago

This should be resolved now with the latest push of the :2.0.21 tag and new entrypoint script. Feel free to re-open if it continues to be a problem.

rounakdatta commented 2 years ago

Thanks for such a quick turnaround, I took a pull of :2.0.21 and indeed the problem is fixed!