tbh/breadcrumb-the-shire
1
0
Fork 0
forked from fa/breadcrumb-the-shire
Code Pull Requests Activity

fix(notifications): use per-type dedup window, 24h for scheduler failures

Browse Source 
Change DEDUPE_TYPES from bool to int|true — int value overrides the
default 30-min window. scheduler.job_failed uses 1440 min (24h) so
admins receive at most one notification per failed job per day.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Felix Schneider
2026-04-06 12:29:04 +02:00
parent 97ff3ce135
commit 34892aff6f
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
modules/notifications/lib/Module/Notifications/Service/NotificationService.php
View File

@@ -9,13 +9,15 @@ class NotificationService
{
private const MESSAGE_META_KEY = '__message';
private const DEDUPE_WINDOW_MINUTES = 30;
/** @var array<string, int|true> Type → dedup window in minutes (true = default window). */
private const DEDUPE_TYPES = [
'user.created' => true,
'user.deleted' => true,
'user.activated' => true,
'user.deactivated' => true,
'user.assignment_changed' => true,
'scheduler.job_failed' => true,
'scheduler.job_failed' => 1440,
];
public function __construct(
@@ -233,6 +235,9 @@ class NotificationService
return [];
}
$windowMinutes = self::DEDUPE_TYPES[$type];
$windowMinutes = is_int($windowMinutes) && $windowMinutes > 0 ? $windowMinutes : self::DEDUPE_WINDOW_MINUTES;
$tenantScope = $tenantId !== null && $tenantId > 0 ? $tenantId : 0;
$raw = implode('|', [
'recipient:' . $recipientUserId,
@@ -243,7 +248,7 @@ class NotificationService
return [
'fingerprint' => hash('sha256', $raw),
'bucket' => intdiv(time(), self::DEDUPE_WINDOW_MINUTES * 60),
'bucket' => intdiv(time(), $windowMinutes * 60),
];
}