Icinga / icinga-web

Icinga Web 1.x, the old new web interface (EOL 31.12.2018)
22 stars 11 forks source link

[dev.icinga.com #12173] Session handler for PHP7 #1378

Closed icinga-migration closed 8 years ago

icinga-migration commented 8 years ago

This issue has been migrated from Redmine: https://dev.icinga.com/issues/12173

Created by mfrosch on 2016-07-18 13:54:46 +00:00

Assignee: mfrosch Status: Resolved (closed on 2016-07-18 16:54:06 +00:00) Target Version: 1.14 Last Update: 2016-07-18 16:54:06 +00:00 (in Redmine)

Icinga Version: 2.4.10
Icinga Web Version: 1.13.1
IDO Version: 2.4.10
OS Version: Debian/stretch
DB Type: MySQL
DB Version: ----
Browser Version: ----

Return values of certain session functions must be bool in PHP 7.

Errors are like:

[Mon Jul 18 10:43:42.747406 2016] [:error] [pid 8435] [client 10.102.1.11:40954] PHP Warning:  session_start(): Session callback expects true/false return value in /usr/share/icinga-web/lib/agavi/src/storage/AgaviSessionStorage.class.php on line 132
[Mon Jul 18 10:43:42.747431 2016] [:error] [pid 8435] [client 10.102.1.11:40954] PHP Fatal error:  session_start(): Failed to initialize storage module: user (path: /var/lib/php/sessions) in /usr/share/icinga-web/lib/agavi/src/storage/AgaviSessionStorage.class.php on line 132
[Mon Jul 18 10:43:42.747472 2016] [:error] [pid 8435] [client 10.102.1.11:40954] PHP Warning:  Unknown: Session callback expects true/false return value in Unknown on line 0

Changesets

2016-07-18 13:57:35 +00:00 by (unknown) 08bd86f64a8835c106c84ce38801fa617afd8e88

AppKitDoctrineSessionStorage: Update for PHP 7 session interfaces

fixes #12173
icinga-migration commented 8 years ago

Updated by mfrosch on 2016-07-18 13:56:41 +00:00

In addition, data to sessionWrite must not be given by reference

icinga-migration commented 8 years ago

Updated by Anonymous on 2016-07-18 16:54:06 +00:00

Applied in changeset 08bd86f64a8835c106c84ce38801fa617afd8e88.