fix: handle LibresignException

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>

Author: vitormattos

lib/Controller/RequestSignatureController.php

@@ -9,6 +9,7 @@
 namespace OCA\Libresign\Controller;
 
 use OCA\Libresign\AppInfo\Application;
+use OCA\Libresign\Exception\LibresignException;
 use OCA\Libresign\Helper\ValidateHelper;
 use OCA\Libresign\Middleware\Attribute\RequireManager;
 use OCA\Libresign\ResponseDefinitions;
@@ -91,10 +92,23 @@ public function request(array $file, array $users, string $name, ?string $callba
 				],
 				Http::STATUS_OK
 			);
+		} catch (LibresignException $e) {
+			$errorMessage = $e->getMessage();
+			$decoded = json_decode($errorMessage, true);
+			if (json_last_error() === JSON_ERROR_NONE && isset($decoded['errors'])) {
+				$errorMessage = $decoded['errors'][0]['message'] ?? $errorMessage;
+			}
+			return new DataResponse(
+				[
+					'message' => $errorMessage,
+				],
+				Http::STATUS_UNPROCESSABLE_ENTITY
+			);
 		} catch (\Throwable $th) {
+			$errorMessage = $th->getMessage();
 			return new DataResponse(
 				[
-					'message' => $th->getMessage(),
+					'message' => $errorMessage,
 				],
 				Http::STATUS_UNPROCESSABLE_ENTITY
 			);

tests/integration/features/sign/request.feature

@@ -222,8 +222,8 @@ Feature: request-signature
       | users | [{"identify":{"account":"signer2"}}] |
       | name | document |
     Then the response should be a JSON array with the following mandatory values
-      | key                   | value           |
-      | (jq).ocs.data.message | User not found. |
+      | key                   | value        |
+      | (jq).ocs.data.message | Invalid user |
 
   Scenario: Request to sign with success using account as identifier
     Given as user "admin"
@@ -273,8 +273,8 @@ Feature: request-signature
       | name | document |
     Then the response should have a status code 422
     Then the response should be a JSON array with the following mandatory values
-      | key                   | value           |
-      | (jq).ocs.data.message | User not found. |
+      | key                   | value        |
+      | (jq).ocs.data.message | Invalid user |
 
   Scenario: Request to sign with error using email as account identifier
     Given as user "admin"
@@ -285,8 +285,8 @@ Feature: request-signature
       | name | document |
     Then the response should have a status code 422
     Then the response should be a JSON array with the following mandatory values
-      | key                   | value           |
-      | (jq).ocs.data.message | User not found. |
+      | key                   | value        |
+      | (jq).ocs.data.message | Invalid user |
 
   Scenario: Request to sign with success using email as identifier and URL as file
     Given as user "admin"