View Issue Details

Jump to NotesJump to History
This bug affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
20083Bug reportsInstallationpublic2025-05-02 18:112025-06-02 10:35
Reporteroxziiii Assigned Togabrieljenik  
PrioritynoneSeveritypartial_block 
Status in code reviewResolutionopen 
Summary20083: Pre-Installation check: Session writable re-check is impossible
Description

I have just set up Limesurvey version limesurvey6.13.1+250428 and failed during the pre-installation "Session writable" check. If the PHP variable "session.save_path" is not set or invalid when entering the pre-installation check, changing it afterwards, restarting PHP FPM, and then either clicking the "recheck" button or reloading the site does not do a thing.

After debugging this for almost an hour, I have grepped for this check in the code and found the cause in application/controllers/InstallerController.php. First, the session and the "saveCheck" field is set in stepLicense(). Then, in stepPreInstallationCheck() this value is checked. However, if one fixes the error and reloads the page, stepLicense() will not be called again as
(Write here what happened instead)accepting the GNU GPL is not required again. Thus, this error cannot go away unless one restarts the whole installation setup.

For context, LimeSurvey runs in a chroot enforced by a custom PHP FPM profile.

Steps To Reproduce

Steps to reproduce

  1. Start with an invalid session.save_path PHP value.
  2. Install a blank LimeSurvey.
  3. Enter the Installation Wizard
  4. Proceed until the Pre-installation check
  5. Get notified about "Session writable" being incorrect
  6. Fix the error, restart PHP FPM or the web server, depending on your setup
  7. Click the recheck button
  8. See the new session.save_path being displayed, but the check still fails

Expected result

I'd expect the "Session writable" check to succeed after fixing the issue and clicking recheck.

Actual result

The error message is updated but the state requires a complete restart of the installation wizard.

TagsNo tags attached.
Bug heat6
Complete LimeSurvey version number (& build)limesurvey6.13.1+250428
I will donate to the project if issue is resolvedNo
Browser
Database type & versionPostgreSQL 17.4
Server OS (if known)OpenBSD current
Webserver software & version (if known)OpenBSD current httpd
PHP VersionPHP 8.2.28

Users monitoring this issue

oxziiii

Activities

gabrieljenik

gabrieljenik

2025-05-30 16:08

manager   ~82793

https://github.com/LimeSurvey/LimeSurvey/pull/4304

Changed the way the session check is done.
The way it was only worked after refreshing 2 times the page.
DenisChenu

DenisChenu

2025-06-02 10:35

developer   ~82797

No way to use a redirect or an ajax call this to call php script 2 times ?

I think an ajax call can do the trick, no ?

Issue History

Date Modified Username Field Change
2025-05-02 18:11 oxziiii New Issue
2025-05-02 18:19 oxziiii Issue Monitored: oxziiii
2025-05-02 18:19 oxziiii Bug heat 0 => 2
2025-05-05 18:14 tibor.pacalat Assigned To => gabrieljenik
2025-05-05 18:14 tibor.pacalat Status new => assigned
2025-05-30 16:08 gabrieljenik Assigned To gabrieljenik => DenisChenu
2025-05-30 16:08 gabrieljenik Status assigned => ready for code review
2025-05-30 16:08 gabrieljenik Note Added: 82793
2025-05-30 16:08 gabrieljenik Bug heat 2 => 4
2025-06-02 10:35 DenisChenu Note Added: 82797
2025-06-02 10:35 DenisChenu Bug heat 4 => 6
2025-06-02 10:35 DenisChenu Assigned To DenisChenu => gabrieljenik
2025-06-02 10:35 DenisChenu Status ready for code review => in code review