AllskyTeam / allsky

A Raspberry Pi operated Wireless Allsky Camera
MIT License
1.16k stars 180 forks source link

3710 upgrade issue with sessions #3789

Closed Alex-developer closed 1 month ago

Alex-developer commented 1 month ago

It seems that the initial permissions on the /var/lib/php/sessions directory still work after an allsky upgrade so we don't need to worry about them.

Test process

1) Install clean copy of bookworm lite 2) Install git 3) Clone allsky 4) Install master branch 5) Set settings and allow it to capture a few images (Its nighttime) 6) Stop allsky and allskyperiodic 7) unmount ~/allsky/tmp 8) Move allsky to allsky-old 9) Clone allsky repo 10) Checkout this branch 11) Install Allsky 12) Save setting (worked fine)

The code checks to make sure that the php sessions are being using the filesystem, the only time this will not be the case is if the user has moved the sessions somewhere else such as redis in which case we don't need to do the check. If any sessions are found to NOT be owned by the web server owner then they are changed

Code has been shellchecked