Checks for empty email before sending notification

Author: LoganTFox

src/Listeners/SendSecurityNotification.php

@@ -15,7 +15,9 @@ public function handle(SecureFieldsUpdated $event): void
     {
         $notificationClass = config('security-notifications.notifications.secure_fields');
 
-        Notification::route('mail', $event->original_email)
-            ->notify(new $notificationClass($event->fields, $event->updated_at));
+        if (! empty($event->original_email)) {
+            Notification::route('mail', $event->original_email)
+                ->notify(new $notificationClass($event->fields, $event->updated_at));
+        }
     }
 }

tests/Unit/Listeners/SendSecurityNotificationTest.php

@@ -30,4 +30,21 @@
             && Arr::has($notification->fields, 'username')
             && Arr::has($notification->fields, 'password');
     });
+});
+
+it('does not send the notification if the email is an empty string', function () {
+    Notification::fake();
+
+    $user = User::factory()->create();
+
+    $event = (new SecureFieldsUpdated(
+        model: $user,
+        fields: [],
+        original_email: '',
+        updated_at: now(),
+    ));
+
+    (new SendSecurityNotification)->handle($event);
+
+    Notification::assertNothingSent();
 });