Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP 7.2 Support? #3185

Closed
freaknils opened this issue Dec 4, 2017 · 9 comments · Fixed by #3315
Closed

PHP 7.2 Support? #3185

freaknils opened this issue Dec 4, 2017 · 9 comments · Fixed by #3315
Milestone

Comments

@freaknils
Copy link

After updating PHP to version 7.2.0 icingaweb2 does not work anymore

Fatal error: Uncaught ErrorException: session_name(): Cannot change session name when session is active in .../icingaweb2/library/Icinga/Web/Session/PhpSession.php:97 Stack trace:
#0 [internal function]: Icinga\Application\ApplicationBootstrap->Icinga\Application\{closure}(2, 'session_name():...', '/usr/share/weba...', 97, Array)
#1 .../icingaweb2/library/Icinga/Web/Session/PhpSession.php(97): session_name('Icingaweb2')
#2 .../icingaweb2/library/Icinga/Web/Session/PhpSession.php(152): Icinga\Web\Session\PhpSession->open()
#3 .../icingaweb2/library/Icinga/Web/Controller/ActionController.php(544): Icinga\Web\Session\PhpSession->write()
#4 .../icingaweb2/library/Icinga/Web/Controller/ActionController.php(489): Icinga\Web\Controller\ActionController->shutdownSession()
#5 .../icingaweb2/library/vendor/Zend/Controller/Action.php(512): Icinga\Web\Controller\ActionController->postDispatch()
#6 .../icingaweb2/library/Icinga/Web in .../icingaweb2/library/vendor/Zend/Controller/Plugin/Broker.php on line 332

Your Environment

:~ > icinga2 --version
icinga2 - The Icinga 2 network monitoring daemon (version: r2.6.3-1)

Copyright (c) 2012-2017 Icinga Development Team (https://www.icinga.com/)
License GPLv2+: GNU GPL version 2 or later <http://gnu.org/licenses/gpl2.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Application information:
  Installation root: /usr
  Sysconf directory: /etc
  Run directory: /run
  Local state directory: /var
  Package data directory: /usr/share/icinga2
  State path: /var/lib/icinga2/icinga2.state
  Modified attributes path: /var/lib/icinga2/modified-attributes.conf
  Objects path: /var/cache/icinga2/icinga2.debug
  Vars path: /var/cache/icinga2/icinga2.vars
  PID path: /run/icinga2/icinga2.pid

System information:
  Platform: Arch Linux ARM
  Platform version: Unknown
  Kernel: Linux
  Kernel version: 4.14.3-1-ARCH
  Architecture: armv7l

Icingaweb2 version is 2.4.1 on archlinux.

Will there be a php 7.2 support soon?

@lippserd
Copy link
Member

lippserd commented Dec 4, 2017

Hi,

Thanks for the report. When does this error occur?

Best regards,
Eric

@freaknils
Copy link
Author

freaknils commented Dec 4, 2017

Hi,

Direct after login. Only white page with this error.
After downgrading to php 7.1.12 everything works well again.

Thanks!

@draeklae
Copy link

draeklae commented Dec 4, 2017

Can confirm this is a problem with icingaweb2 version 2.4.2 and PHP 7.2.0.

Can also confirm the downgrade fixes it.

@lippserd
Copy link
Member

lippserd commented Dec 5, 2017

@draeklae Are you also running Arch Linux?

@lippserd
Copy link
Member

lippserd commented Dec 5, 2017

I think this is not related to the PHP build on Arch Linux. I reproduced it with Remi's php72 package on CentOS 7.

@draeklae
Copy link

draeklae commented Dec 5, 2017

@lippserd Arch Linux ARM. AFAIK the PHP packages are identical to the Arch Linux base though.

@cschug
Copy link

cschug commented Dec 7, 2017

It should be noted that with PHP 7.2.0 there was a considerable amount of changes regarding session handling, see php/php-src@7f196e3 for example.

@egnappahz
Copy link

egnappahz commented Dec 7, 2017

FYI: I have exactly the same stated error, im using a regular(x86_64) base arch linux which is up to date. (so thats also php 7.2 and icingaweb2 2.4.2-1)

@Al2Klimov
Copy link
Member

This will be fixed once #3186 has been merged.

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

Successfully merging a pull request may close this issue.

6 participants