Improve comma-separated email addresses

Author: bencroker

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Release Notes for Sherlock
 
+## 5.1.2 - 2025-04-11
+
+- Improved the processing of comma-separated notification email addresses ([#46](https://github.com/putyourlightson/craft-sherlock/issues/46)).
+
 ## 5.1.1 - 2025-04-09
 
 - Fixed a bug in which notification emails were not being sent on newly failed scans ([#46](https://github.com/putyourlightson/craft-sherlock/issues/46)).

composer.json

@@ -1,7 +1,7 @@
 {
   "name": "putyourlightson/craft-sherlock",
   "description": "Security scanner and monitor to keep your site and CMS secure.",
-  "version": "5.1.1",
+  "version": "5.1.2",
   "type": "craft-plugin",
   "homepage": "https://putyourlightson.com/plugins/sherlock",
   "license": "proprietary",

src/services/ScansService.php

@@ -7,6 +7,7 @@
 
 use Craft;
 use craft\base\Component;
+use craft\helpers\StringHelper;
 use craft\helpers\UrlHelper;
 use putyourlightson\sherlock\models\ScanModel;
 use putyourlightson\sherlock\records\ScanRecord;
@@ -209,8 +210,10 @@ private function sendNotifications(ScanModel $scanModel): void
      */
     private function sendLogNotificationEmail(string $subject, string $body, string $log): void
     {
+        $to = StringHelper::split(Sherlock::$plugin->settings->notificationEmailAddresses);
+
         Craft::$app->mailer->compose()
-            ->setTo(Sherlock::$plugin->settings->notificationEmailAddresses)
+            ->setTo($to)
             ->setSubject(Craft::$app->getSites()->getCurrentSite()->name . ' - ' . $subject)
             ->setHtmlBody($body . UrlHelper::cpUrl('sherlock'))
             ->send();