simplesamlphp / simplesamlphp-module-statistics

hooks/hook_cron.php

<?php

declare(strict_types=1);

use Exception;
use SimpleSAML\Assert\Assert;
use SimpleSAML\Configuration;
use SimpleSAML\Logger;
use SimpleSAML\Module\statistics\Aggregator;

/**
 * Hook to run a cron job.
 *
 * @param array &$croninfo  Output
 */
function statistics_hook_cron(array &$croninfo): void
{
    Assert::keyExists($croninfo, 'summary');
    Assert::keyExists($croninfo, 'tag');

    $statconfig = Configuration::getConfig('module_statistics.php');

    if (is_null($statconfig->getOptionalValue('cron_tag', null))) {
        return;
    }
    if ($statconfig->getOptionalValue('cron_tag', null) !== $croninfo['tag']) {
        return;
    }

    $maxtime = $statconfig->getOptionalInteger('time_limit', null);
    if ($maxtime) {

The expression $maxtime of type integer|null is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== null instead.

        set_time_limit($maxtime);
    }

    try {
        $aggregator = new Aggregator();
        $results = $aggregator->aggregate();
        if (empty($results)) {
            Logger::notice('Output from statistics aggregator was empty.');
        } else {
            $aggregator->store($results);
        }
    } catch (Exception $e) {
        $message = 'Loganalyzer threw exception: ' . $e->getMessage();
        Logger::warning($message);
        $croninfo['summary'][] = $message;
    }
}