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

[8.2 RC2] White Screen of Death when Accessing Admin Page #19793

Closed
MTRichards opened this issue Oct 14, 2015 · 17 comments · Fixed by #19795
Closed

[8.2 RC2] White Screen of Death when Accessing Admin Page #19793

MTRichards opened this issue Oct 14, 2015 · 17 comments · Fixed by #19795

Comments

@MTRichards
Copy link
Contributor

When I log in as an admin and I try to get to the admin page via the top right menu (Admin option) I get a white screen of death.

Expected behavior:
Click on the admin panel and be able to see the admin page, and manage apps that are enabled.

What actually Happened:
Click on the admin panel and get a white screen of death. Log file below.

Likely this is the LDAP app, because as soon as I disable the LDAP app, the white screen goes away and I can access settings on the server.

Log Entry:
image

Setup:
SLES
MySQL
Windows 10 Firefox (repeated with Windows 8.1 Firefox and Chrome)
ownCloud 8.2.0 RC2 released 10/14/2015.
Dropped this onto the SLES enterprise appliance base build, and everything else seems to be working except this.

@MTRichards MTRichards added this to the 8.2-current milestone Oct 14, 2015
@LukasReschke
Copy link
Member

@MTRichards Bonus points for pasting the log data instead of unsearchable screenshots 🙊

@LukasReschke
Copy link
Member

Also could you take a look at the PHP error log? /var/log/apache2/error.log or so?

@RobinMcCorkell
Copy link
Member

Looks like it might be https://github.com/owncloud/core/blob/master/apps/user_ldap/lib/configuration.php#L153, which is invalid syntax on PHP <5.6 (I think). Basically, empty() must be run on a variable, and not on a function result. Fun fact though: if(empty($var)) is equivalent to if(!$var)

cc @blizzz

@MTRichards
Copy link
Contributor Author

I know, right? I am really helpful...
The issue is clear in the last line of the log, so no real need to search.

There is Nothing in the error_log for PHP

@LukasReschke
Copy link
Member

@DeepDiver1975 Don't we have PHP 5.5 execution for master as well on CI?

@MTRichards
Copy link
Contributor Author

PHP info on this server (note: text and searchable):
PHP Version 5.4.20

System Linux eval-appliance-oc8ee-20150915-813-1 3.11.10-29-default #1 SMP Thu Mar 5 16:24:00 UTC 2015 (338c513) x86_64
Server API Apache 2.0 Handler
Virtual Directory Support disabled
Configuration File (php.ini) Path /etc/php5/apache2
Loaded Configuration File /etc/php5/apache2/php.ini
Scan this dir for additional .ini files /etc/php5/conf.d
Additional .ini files parsed /etc/php5/conf.d/apcu.ini, /etc/php5/conf.d/ctype.ini, /etc/php5/conf.d/curl.ini, /etc/php5/conf.d/dom.ini, /etc/php5/conf.d/fileinfo.ini, /etc/php5/conf.d/ftp.ini, /etc/php5/conf.d/gd.ini, /etc/php5/conf.d/gmp.ini, /etc/php5/conf.d/iconv.ini, /etc/php5/conf.d/json.ini, /etc/php5/conf.d/ldap.ini, /etc/php5/conf.d/libsmbclient.ini, /etc/php5/conf.d/mbstring.ini, /etc/php5/conf.d/mysql.ini, /etc/php5/conf.d/mysqli.ini, /etc/php5/conf.d/odbc.ini, /etc/php5/conf.d/openssl.ini, /etc/php5/conf.d/pdo.ini, /etc/php5/conf.d/pdo_mysql.ini, /etc/php5/conf.d/pdo_odbc.ini, /etc/php5/conf.d/posix.ini, /etc/php5/conf.d/soap.ini, /etc/php5/conf.d/xmlreader.ini, /etc/php5/conf.d/xmlwriter.ini, /etc/php5/conf.d/zendguardloader.ini, /etc/php5/conf.d/zip.ini, /etc/php5/conf.d/zlib.ini
PHP API 20100412
PHP Extension 20100525
Zend Extension 220100525
Zend Extension Build API220100525,NTS
PHP Extension Build API20100525,NTS
Debug Build no
Thread Safety disabled
Zend Signal Handling disabled
Zend Memory Manager enabled
Zend Multibyte Support provided by mbstring
IPv6 Support enabled
DTrace Support disabled
Registered PHP Streams php, file, glob, data, http, ftp, https, ftps, zip, compress.zlib
Registered Stream Socket Transports tcp, udp, unix, udg, ssl, sslv3, sslv2, tls
Registered Stream Filters string.rot13, string.toupper, string.tolower, string.strip_tags, convert., consumed, dechunk, convert.iconv., zlib.*

@LukasReschke
Copy link
Member

@blizzz FYI

@karlitschek
Copy link
Contributor

Arggghhh. Thanks @MTRichards for testing this. We still support 5.4

@LukasReschke
Copy link
Member

Arggghhh. Thanks @MTRichards for testing this. We still support 5.4

On a related note this could have been detected automatically. @cmonteroluque this is related to my mail (from some time ago) about manual testing before a release.

PHPStorm is quite able to spot this:

2015-10-14_22-32-45

Can we please make sure that this gets added to the release checklist? We can't just really hope that somebody will hopefully trigger the incompatible code path on an incompatible version in the RC phase. This happened too often already.

LukasReschke added a commit that referenced this issue Oct 14, 2015
Those are only allowed in PHP 5.5, thus making our code incompatible with PHP 5.4

Fixes #19793
@LukasReschke
Copy link
Member

=> #19795

LukasReschke added a commit that referenced this issue Oct 14, 2015
Those are only allowed in PHP 5.5, thus making our code incompatible with PHP 5.4

Fixes #19793
@ghost
Copy link

ghost commented Oct 14, 2015

@LukasReschke got it. Thanks!

LukasReschke added a commit that referenced this issue Oct 14, 2015
Let's see if this helps for cases such as #19793
@MorrisJobke
Copy link
Contributor

This one is fixed in stable8.2: #19796

@nickvergessen
Copy link
Contributor

Reopening until it is fixed in 9.0

@nickvergessen nickvergessen reopened this Oct 15, 2015
@MorrisJobke
Copy link
Contributor

Reopening until it is fixed in 9.0

It's only a merge away ;)

@ghost
Copy link

ghost commented Oct 15, 2015

ok, moving it to 9.0

@ghost ghost modified the milestones: 9.0-next, 8.2-current Oct 15, 2015
@LukasReschke
Copy link
Member

@cmonteroluque if you press merge at "#19797" we can also close this ;)

@MorrisJobke
Copy link
Contributor

Was merged by @DeepDiver1975 -> I will move this to 8.2 so that we know that the fix for this is in 8.2 and upwards and not think that it only was included in 9.0 (there is no brown label for the .0 releases and I don't want to introduce it for one ticket.

@MorrisJobke MorrisJobke modified the milestones: 8.2-current, 9.0-next Oct 15, 2015
DeepDiver1975 pushed a commit to owncloud/user_ldap that referenced this issue Sep 6, 2016
Those are only allowed in PHP 5.5, thus making our code incompatible with PHP 5.4

Fixes owncloud/core#19793
@lock lock bot locked as resolved and limited conversation to collaborators Aug 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants