-
-
Notifications
You must be signed in to change notification settings - Fork 460
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
I get a white screen when i activate this plugin #650
Comments
Try adding |
I've got this error when updated |
500 internal server errors are often difficult to figure out. https://www.google.com/search?q=500+internal+server+error My test site have 768M allocated to memory. Most of these errors are related to htaccess files. |
Line 39 of Singleton is a call to the PHP function I’m looking at ways to limit this but likely won’t be able to test until tonight. |
For now you can comment out https://github.com/afragen/github-updater/blob/develop/src/Singleton.php#L39 until I can fix and test. This will likely cause several non-fatal PHP errors. |
Can confirm I also get the same error with a fresh install of WordPress 4.9.4 on Ubuntu 16.04. Removing the |
Thanks @reddog. I believe the |
Update to latest |
Same error with last develop branch version (line 41), switched to 7.5.0 as suggested by @reddog and now it's working again... |
So the problem is that there is really no difference in Singleton.php from the develop branch to 7.5.0 I realize that this is where the error is firing but can people share anything else that they did to there installations in the same timeframe? Does this issue persist in a local installation? |
Can people provide server details. PHP version, WP version, PHP memory available? |
Is there anything new? I’ve been unable to replicate this issue, does it still exist? |
Still exists. WP 4.9.4; Ubuntu 16.04 LTS; PHP 7.0.27; PHP Memory Limit 256MB. |
Can we all test in a local environment. I’d like to try and replicate this but haven’t been able to. Test using default theme and only with the GitHub Updater plugin active. |
Can you rename |
Are there any mu-plugins? |
@tomkenkel does switching to v7.5.0 work for you? |
I renamed the directory. No fix. |
@tomkenkel I have to say I don’t really understand why 7.5.0 works but develop doesn’t. The Singleton class isn’t significantly different between the 2 versions. |
I’m not good at PHP, but I don't think it's the class. Replacing the Singleton class in develop with the class from 7.5.0 creates a different error: |
The odd thing is there’s not a significant amount of difference in the develop branch. Can you add the following in wp-config.php and check the debug.log in wp-content? define( 'WP_DEBUG', true ); |
Interesting, a new error. Can you change the timeout? https://www.fdgweb.com/increase-php-timeout-values-apache-ngnix/ And I have zero idea why there would be a timeout here. Not even hitting an API. You might try adding the filter to bypass wp-cron and eliminate that as an issue.
|
Longer timeout [04-Mar-2018 23:05:18 UTC] PHP Fatal error: Maximum execution time of 240 seconds exceeded in /var/www/public/site/wp-content/plugins/github-updater/src/Singleton.php on line 41 [04-Mar-2018 23:06:41 UTC] PHP Fatal error: Maximum execution time of 240 seconds exceeded in /var/www/public/site/wp-content/plugins/github-updater/src/Singleton.php on line 42 [04-Mar-2018 23:10:08 UTC] PHP Fatal error: Maximum execution time of 240 seconds exceeded in /var/www/public/site/wp-content/plugins/github-updater-develop/src/Singleton.php on line 41 |
Set the timeout for 600, humor me. Also, if you could dump the $backtrace variable from that line to a gist, that would help. |
Another thing to try is changing the line to the following.
|
Odd that a standard PHP function would timeout. |
Can you try the branch |
Different error
This was run on Scotchbox’s recently updated server stack. I can try tonight with the stack I used when we started. |
Is there more in the error logs? This error doesn't seem to trace back to GitHub Updater directly, does it? |
I think I may have found an fixed this error. I think it may be related to #656 Please test and let me know. |
Installs and activates as expected. I was able to install and activate a theme from GitHub. I haven’t tried updating the theme. I started getting the cron warning below after updating Scotchbox. It’s always something. Thanks for your persistence! |
That’s just a warning that WP-Cron hasn’t run in over 24 hours. If this persists it may mean that something in the server setup is interfering with how WP-Cron runs and you might need to create an actual cron job. Look at the wiki regarding background processing. |
@roger2014jr @teolaz @reddog @picasso please let me know if this issue is fixed for you. Thanks. |
@tomkenkel you can also turn off background processing with a filter to see if the issue is WP-Cron only.
|
@afragen updated to 7.5.0, cannot get again the same error on different sites :S |
The error was on the develop branch, not the release. |
This is fixed in latest |
I'm using the latest WordPress with a multi-site install, Now I'm stuck and have to figure out how to uninstall this plugin manually
The text was updated successfully, but these errors were encountered: