Bugfixes

LibreSign · Feb 14, 2021 · 11d887f · 11d887f
1 parent 640d023
commit 11d887f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/lib/Controller/WebhookController.php b/lib/Controller/WebhookController.php
@@ -53,6 +53,8 @@ public function register(array $file, array $users, string $name, ?string $callb
 		];
 		try {
 			$this->webhook->validate($data);
+			$return = $this->webhook->save($data);
+			$this->mail->notifyAllUnsigned();
 		} catch (\Throwable $th) {
 			return new JSONResponse(
 				[
@@ -61,8 +63,6 @@ public function register(array $file, array $users, string $name, ?string $callb
 				Http::STATUS_UNPROCESSABLE_ENTITY
 			);
 		}
-		$return = $this->webhook->save($data);
-		$this->mail->notifyAllUnsigned();
 		return new JSONResponse(
 			[
 				'message' => $this->l10n->t('Success'),

diff --git a/lib/Service/MailService.php b/lib/Service/MailService.php
@@ -92,9 +92,7 @@ public function notifyUnsignedUser(FileUser $data) {
 		try {
 			$this->mailer->send($message);
 		} catch (\Exception $e) {
-			$this->logger->error($e, [
-				'message' => 'Notify unsigned notification mail could not be sent: ' . $e->getMessage()
-			]);
+			$this->logger->error('Notify unsigned notification mail could not be sent: ' . $e->getMessage());
 			throw new LibresignException('Notify unsigned notification mail could not be sent', 1);
 		}
 	}