View Issue Details

This bug affects 1 person(s).
 10
IDProjectCategoryView StatusLast Update
20083Bug reportsInstallationpublic2025-06-23 15:42
Reporteroxziiii Assigned Totibor.pacalat  
PrioritynoneSeveritypartial_block 
Status closedResolutionfixed 
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 heat10
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 ?

gabrieljenik

gabrieljenik

2025-06-05 20:40

manager   ~82836

New approach: https://github.com/LimeSurvey/LimeSurvey/pull/4315

  • Added a Precheck Prepare page where the session is seeded, for later verification of the process.
  • When refreshing the precheck page, based on a cookie, we will navigate back to the prepartion page, to seed the session again.
guest

guest

2025-06-23 14:30

viewer   ~82922

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=38937

LimeBot

LimeBot

2025-06-23 15:42

administrator   ~82929

Fixed in Release 6.15.0+250623

Related Changesets

LimeSurvey: master cb66a011

2025-06-23 14:20

Gabriel Jenik

Committer: GitHub


Details Diff
Fixed issue 20083: Pre-Installation check - Session writable re-check is impossible (04315)

Co-authored-by: lapiudevgit <devgit@lapiu.biz>
Affected Issues
20083
mod - application/controllers/InstallerController.php Diff File
mod - application/views/installer/dbconfig_view.php Diff File
mod - application/views/installer/precheck_view.php Diff File

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
2025-06-05 20:40 gabrieljenik Note Added: 82836
2025-06-06 12:27 DenisChenu Assigned To gabrieljenik => tibor.pacalat
2025-06-06 12:27 DenisChenu Status in code review => ready for testing
2025-06-23 14:21 tibor.pacalat Status ready for testing => resolved
2025-06-23 14:21 tibor.pacalat Resolution open => fixed
2025-06-23 14:30 Changeset attached => LimeSurvey master cb66a011
2025-06-23 14:30 guest Note Added: 82922
2025-06-23 14:30 guest Bug heat 6 => 8
2025-06-23 15:42 LimeBot Note Added: 82929
2025-06-23 15:42 LimeBot Status resolved => closed
2025-06-23 15:42 LimeBot Bug heat 8 => 10