Merge pull request #4254 from LibreSign/backport/4239/stable29

[stable29] feat: validate from uploaded file
LibreSign · Jan 10, 2025 · aa92e9d · aa92e9d
2 parents ff269c5 + f057a6e
commit aa92e9d
Show file tree

Hide file tree

Showing 45 changed files with 2,449 additions and 414 deletions.
diff --git a/.github/workflows/behat-mariadb.yml b/.github/workflows/behat-mariadb.yml
@@ -68,6 +68,9 @@ jobs:
           # Split and keep last
           echo "APP_NAME=${GITHUB_REPOSITORY##*/}" >> $GITHUB_ENV
 
+      - name: Install system dependencies
+        run: sudo apt install poppler-utils
+
       - name: Checkout server
         uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
         with:

diff --git a/.github/workflows/behat-mysql.yml b/.github/workflows/behat-mysql.yml
@@ -68,6 +68,9 @@ jobs:
           # Split and keep last
           echo "APP_NAME=${GITHUB_REPOSITORY##*/}" >> $GITHUB_ENV
 
+      - name: Install system dependencies
+        run: sudo apt install poppler-utils
+
       - name: Checkout server
         uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
         with:

diff --git a/.github/workflows/behat-pgsql.yml b/.github/workflows/behat-pgsql.yml
@@ -69,6 +69,9 @@ jobs:
           # Split and keep last
           echo "APP_NAME=${GITHUB_REPOSITORY##*/}" >> $GITHUB_ENV
 
+      - name: Install system dependencies
+        run: sudo apt install poppler-utils
+
       - name: Checkout server
         uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
         with:

diff --git a/.github/workflows/behat-sqlite.yml b/.github/workflows/behat-sqlite.yml
@@ -60,6 +60,9 @@ jobs:
           # Split and keep last
           echo "APP_NAME=${GITHUB_REPOSITORY##*/}" >> $GITHUB_ENV
 
+      - name: Install system dependencies
+        run: sudo apt install poppler-utils
+
       - name: Checkout server
         uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
         with:

diff --git a/.github/workflows/phpunit-mariadb.yml b/.github/workflows/phpunit-mariadb.yml
@@ -87,6 +87,9 @@ jobs:
           # Split and keep last
           echo "APP_NAME=${GITHUB_REPOSITORY##*/}" >> $GITHUB_ENV
 
+      - name: Install system dependencies
+        run: sudo apt install poppler-utils
+
       - name: Checkout server
         uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
         with:

diff --git a/.github/workflows/phpunit-mysql.yml b/.github/workflows/phpunit-mysql.yml
@@ -87,6 +87,9 @@ jobs:
           # Split and keep last
           echo "APP_NAME=${GITHUB_REPOSITORY##*/}" >> $GITHUB_ENV
 
+      - name: Install system dependencies
+        run: sudo apt install poppler-utils
+
       - name: Checkout server
         uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
         with:

diff --git a/.github/workflows/phpunit-pgsql.yml b/.github/workflows/phpunit-pgsql.yml
@@ -88,6 +88,9 @@ jobs:
           # Split and keep last
           echo "APP_NAME=${GITHUB_REPOSITORY##*/}" >> $GITHUB_ENV
 
+      - name: Install system dependencies
+        run: sudo apt install poppler-utils
+
       - name: Checkout server
         uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
         with:

diff --git a/.github/workflows/phpunit-sqlite.yml b/.github/workflows/phpunit-sqlite.yml
@@ -78,6 +78,9 @@ jobs:
           # Split and keep last
           echo "APP_NAME=${GITHUB_REPOSITORY##*/}" >> $GITHUB_ENV
 
+      - name: Install system dependencies
+        run: sudo apt install poppler-utils
+
       - name: Checkout server
         uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
         with:

diff --git a/appinfo/info.xml b/appinfo/info.xml
@@ -25,7 +25,7 @@ Developed with ❤️ by [LibreCode](https://librecode.coop). Help us transform
 
 * [Donate with GitHub Sponsor: ![Donate using GitHub Sponsor](https://img.shields.io/static/v1?label=Sponsor&message=%E2%9D%A4&logo=GitHub&color=%23fe8e86)](https://github.com/sponsors/libresign)
     ]]></description>
-    <version>9.7.4</version>
+    <version>9.8.0</version>
     <licence>agpl</licence>
     <author mail="[email protected]" homepage="https://librecode.coop">LibreCode</author>
     <documentation>

diff --git a/composer.json b/composer.json
@@ -7,6 +7,7 @@
 		"mikehaertl/php-pdftk": "^0.13.0",
 		"mpdf/mpdf": "^8.2",
 		"pagerfanta/pagerfanta": "^3.6",
+		"phpseclib/phpseclib": "^3.0",
 		"smalot/pdfparser": "^2.4",
 		"wobeto/email-blur": "^1.0"
 	},

diff --git a/composer.lock b/composer.lock
diff --git a/lib/Command/Configure/Check.php b/lib/Command/Configure/Check.php
@@ -82,10 +82,12 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
 		if (count($result)) {
 			$table = new Table($output);
+			$table->setColumnMaxWidth(3, 40);
 			foreach ($result as $row) {
 				$table->addRow([
 					new TableCell($row->getStatus(), ['style' => new TableCellStyle([
-						'bg' => $row->getStatus() === 'success' ? 'green' : 'red',
+						'bg' => $this->getStatusColor($row->getStatus()),
+						'fg' => 'black',
 						'align' => 'center',
 					])]),
 					$row->getResource(),
@@ -105,4 +107,13 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 		}
 		return 0;
 	}
+
+	private function getStatusColor($status): string {
+		return match ($status) {
+			'success' => 'green',
+			'error' => 'red',
+			'info' => 'bright-yellow',
+			default => 'red',
+		};
+	}
 }