Skip to content
Snippets Groups Projects

fix: :bug: Fix SQL error for MySQL vs PGSQL in Challenges delete

Merged Ghost User requested to merge fix_challenges_sql into main
2 files
+ 20
1
Compare changes
  • Side-by-side
  • Inline
Files
2
@@ -4,6 +4,8 @@ declare(strict_types=1);
namespace SimpleSAML\Module\perun\databaseCommand;
use SimpleSAML\Module\perun\Exception;
class ChallengesDbCmd extends DatabaseCommand
{
private const CHALLENGES_TABLE_NAME = 'scriptChallenges';
@@ -64,8 +66,15 @@ class ChallengesDbCmd extends DatabaseCommand
public function deleteOldChallenges(): bool
{
if ($this->isMysql()) {
$filter = ' < (NOW() - INTERVAL 5 MINUTE)';
} elseif ($this->isPgsql()) {
$filter = ' < (NOW() - INTERVAL \'5 MINUTES\')';
} else {
throw new \Exception("Unrecognized driver for database");
}
$query = 'DELETE FROM ' . self::CHALLENGES_TABLE_NAME . ' WHERE '
. self::DATE_COLUMN . ' < (NOW() - INTERVAL 5 MINUTE)';
. self::DATE_COLUMN . $filter;
$params = [];
Loading