Merge remote-tracking branch 'origin/feature/190-maestro-notification-exception-handling' into release/selvbetjening-4.6.0

Author: jekuaitk

CHANGELOG.md

@@ -15,6 +15,8 @@ before starting to add changes. Use example [placed in the end of the page](#exa
   - Added support for MeMo 1.2 and added additional validation of MeMo actions.
 - [PR-202](https://github.com/OS2Forms/os2forms/pull/202)
   - Removed non-digits from recipient id in Maestro digital post notifications.
+- [PR-191](https://github.com/OS2Forms/os2forms/pull/191)
+  - Re-throws exception to ensure failed status during Maestro notification job.
 
 ## [4.1.0] 2025-06-03
 

modules/os2forms_forloeb/src/MaestroHelper.php

@@ -205,7 +205,14 @@ public function processJob(Job $job): JobResult {
 
     $submission = $this->webformSubmissionStorage->load($submissionID);
 
-    $this->sendNotification($notificationType, $submission, $templateTask, $maestroQueueID);
+    try {
+      $this->sendNotification($notificationType, $submission, $templateTask, $maestroQueueID);
+    }
+    catch (\Exception $e) {
+      // Logging is done by the sendNotification method.
+      // The job should be considered failed.
+      return JobResult::failure($e->getMessage());
+    }
 
     return JobResult::success();
   }
@@ -261,12 +268,15 @@ private function sendNotification(
       }
     }
     catch (\Exception $exception) {
+      // Log with context and rethrow exception.
       $this->error('Error sending notification: @message', $context + [
         '@message' => $exception->getMessage(),
         'handler_id' => 'os2forms_forloeb',
         'operation' => 'notification failed',
         'exception' => $exception,
       ]);
+
+      throw $exception;
     }
   }