-
Notifications
You must be signed in to change notification settings - Fork 417
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
Too many checking_for_updates requests #572
Comments
An old version had a bug that could cause excessive update requests, but that was fixed long ago. It should no longer happen with version 5.4. Could it be that the customers in question have a very old version of your plugin that is using a very old version of PUC? Also, it looks like the screenshot did not upload properly. Edit: You can control how often PUC checks for updates in multiple ways, but that will only help if the extra requests are not caused by a bug or an outdated PUC version. For example, you could use the $updateChecker->addFilter(
'check_now',
function ($shouldCheck, $lastCheckTimestamp, $checkPeriod) {
if ( should_stop_this_request() ) {
return false;
}
return $shouldCheck;
},
10,
3
); |
Thank you for your response. Updated my initial post with screenshot. |
Well, I'm not aware of any current bugs or conflicts that could cause this kind of issue. Some other ideas, mostly not very plausible:
If you have access to the customer's site, some of the "storage issues" possibilities can be tested by using the Debug Bar plugin. The "PUC (your-slug)" section should show the last check timestamp. You can test if the timestamp changes after you check for updates (you'll need to refresh the page to refresh the debug info). |
@YahnisElsts If you have a moment, could you identify for me the commit that fixes the issue? Having seen it in my logs for years (but never found the cause because of the difficulty of accessing an end user site who has it), I'm very curious! |
Unfortunately, it was years ago and I no longer remember what the original issue was. I spent some time looking through the history (including from before PUC was migrated to GitHub) and found 5aee0d7 where a Cron-related bug in another plugin apparently caused me a lot of trouble. However, I'm not sure if that's really the one. |
Thanks for looking! |
Hi,
For the most part we see that update check requests are coming once or twice a day, but a couple of our customers are sending thousands of these requests each day. I'm attaching a screenshot.
Is this a known issue and if it is, are there any plans in resolving it?
Or maybe there is something that we can do to limit it via our plugin?
We are suing Plugin Update Checker Library 5.4.
The text was updated successfully, but these errors were encountered: