<?php
/**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2023, the Friendica project
*
* @license GNU AGPL version 3 or any later version
*
use Friendica\Core\Logger;
use Friendica\Core\Worker;
use Friendica\Database\DBA;
-use Friendica\Model\GServer;
+use Friendica\DI;
+use Friendica\Util\DateTimeFormat;
+use Friendica\Util\Strings;
class UpdateGServers
{
/**
- * Updates the first 250 servers
+ * Updates a defined number of servers
*/
public static function execute()
{
- $gservers = DBA::p("SELECT `url`, `created`, `last_failure`, `last_contact` FROM `gserver` ORDER BY rand()");
- if (!DBA::isResult($gservers)) {
+ $update_limit = DI::config()->get('system', 'gserver_update_limit');
+ if (empty($update_limit)) {
return;
}
- $updated = 0;
+ $updating = Worker::countWorkersByCommand('UpdateGServer');
+ $limit = $update_limit - $updating;
+ if ($limit <= 0) {
+ Logger::info('The number of currently running jobs exceed the limit');
+ return;
+ }
- while ($gserver = DBA::fetch($gservers)) {
- if (!GServer::updateNeeded($gserver['created'], '', $gserver['last_failure'], $gserver['last_contact'])) {
- continue;
- }
- Logger::info('Update server status', ['server' => $gserver['url']]);
+ $total = DBA::count('gserver');
+ $condition = ["NOT `blocked` AND `next_contact` < ? AND (`nurl` != ? OR `url` != ?)", DateTimeFormat::utcNow(), '', ''];
+ $outdated = DBA::count('gserver', $condition);
+ Logger::info('Server status', ['total' => $total, 'outdated' => $outdated, 'updating' => $limit]);
- Worker::add(PRIORITY_LOW, 'UpdateGServer', $gserver['url']);
+ $gservers = DBA::select('gserver', ['url', 'nurl'], $condition, ['limit' => $limit]);
+ if (!DBA::isResult($gservers)) {
+ return;
+ }
- if (++$updated > 250) {
- return;
+ $count = 0;
+ while ($gserver = DBA::fetch($gservers)) {
+ // Sometimes the "nurl" and "url" doesn't seem to fit, see https://forum.friendi.ca/display/ec054ce7-155f-c94d-6159-f50372664245
+ // There are duplicated "url" but not "nurl". So we check both addresses instead of just overwriting them,
+ // since that would mean loosing data.
+ if (!empty($gserver['url'])) {
+ if (UpdateGServer::add(Worker::PRIORITY_LOW, $gserver['url'])) {
+ $count++;
+ }
+ }
+ if (!empty($gserver['nurl']) && ($gserver['nurl'] != Strings::normaliseLink($gserver['url']))) {
+ if (UpdateGServer::add(Worker::PRIORITY_LOW, $gserver['nurl'])) {
+ $count++;
+ }
}
+ Worker::coolDown();
}
+ DBA::close($gservers);
+ Logger::info('Updated servers', ['count' => $count]);
}
}