dst/store
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: update SendPushNotificationJob to correctly handle expired subscriptions; modify PushSubscriptionService to ensure user_id is set during subscription creation

Browse Source
This commit is contained in:
Yoga Pangestu 2026-07-05 13:33:18 +07:00
parent 5a5d2c6f78
commit 18ccd7a3c0
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/Jobs/SendPushNotificationJob.php
View File

@ -84,8 +84,8 @@ private function sendWebPush(): void
$expiredEndpoints = []; $expiredEndpoints = [];
foreach ($webPush->flush() as $report) { foreach ($webPush->flush() as $report) {
if (! $report->isSuccess()) { if ($report->isSubscriptionExpired()) {
$expiredEndpoints[] = (string) $report->getRequest()->getUri(); $expiredEndpoints[] = $report->getEndpoint();
} }
} }

2
app/Services/System/PushSubscriptionService.php
View File

@ -10,10 +10,10 @@ public function updateOrCreateSubscription(array $validated, int $userId): void
{ {
PushSubscription::updateOrCreate( PushSubscription::updateOrCreate(
[ [
'user_id' => $userId,
'endpoint' => $validated['endpoint'], 'endpoint' => $validated['endpoint'],
], ],
[ [
'user_id' => $userId,
'public_key' => $validated['publicKey'], 'public_key' => $validated['publicKey'],
'auth_token' => $validated['authToken'], 'auth_token' => $validated['authToken'],
] ]