kalaksi / docker-phpldapadmin

Docker container for phpLDAPadmin.
MIT License
7 stars 3 forks source link

PHP sessions folder is not cleaned up #2

Open ypapouin opened 1 year ago

ypapouin commented 1 year ago

My phpldapadmin isntance is up since Jan 2023. Today I had a out of space failure on the machine, Looking at the container data, it looks like php sessions files are never cleaned up.

ncdu 1.15.1 ~ Use the arrow keys to navigate, press ? for help --- /var/lib/docker/overlay2/032375f0a0c54f521906bdf8ba3ccef03f2423f895e91a48aa85fa9e6b813a57/merged/var/lib/php ----------------------------------------------------------------------------------------------- /.. 14.2 GiB [##########] /sessions 28.0 KiB [ ] /modules

ypapouin commented 1 year ago

My temporary workaround is to delete sessions files older than 30 days using a cron task: docker exec --user root phpldapadmin-phpldapadmin-1 find /var/lib/php/sessions -mtime +30 -type f -delete

kalaksi commented 1 year ago

Oh, wow. Looks like session garbage collection can be controlled through php.ini, but Debian has it disabled by default and uses a cron-job for cleaning (which doesn't work in this case).

The solution seems to be to simply enable it in php.ini, so I'll have the fix out soon.

kalaksi commented 1 year ago

I don't have an easy way to test that the session GC actually works, so could you help and confirm that it works for you?