Skip to content
This repository has been archived by the owner on Jan 15, 2019. It is now read-only.

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

Closed
icinga-migration opened this issue Jul 18, 2016 · 2 comments
Closed

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

icinga-migration opened this issue Jul 18, 2016 · 2 comments
Milestone

Comments

@icinga-migration
Copy link

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) 08bd86f

AppKitDoctrineSessionStorage: Update for PHP 7 session interfaces

fixes #12173
@icinga-migration
Copy link
Author

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
Copy link
Author

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

  • Status changed from New to Resolved
  • Done % changed from 0 to 100

Applied in changeset 08bd86f.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant