Merge pull request #2260 from LibreSign/bugfixes/app-files

Bugfixes app files

lib/AppInfo/Application.php

@@ -32,6 +32,7 @@
 use OCA\Libresign\Listener\LoadSidebarListener;
 use OCA\Libresign\Listener\NotificationListener;
 use OCA\Libresign\Listener\SignedListener;
+use OCA\Libresign\Middleware\GlobalInjectionMiddleware;
 use OCA\Libresign\Middleware\InjectionMiddleware;
 use OCA\Libresign\Notification\Notifier;
 use OCP\AppFramework\App;
@@ -61,6 +62,7 @@ public function boot(IBootContext $context): void {
 	}
 
 	public function register(IRegistrationContext $context): void {
+		$context->registerMiddleWare(GlobalInjectionMiddleware::class, true);
 		$context->registerMiddleWare(InjectionMiddleware::class);
 
 		$context->registerNotifierService(Notifier::class);

lib/Middleware/GlobalInjectionMiddleware.php

@@ -0,0 +1,42 @@
+<?php
+
+declare(strict_types=1);
+/**
+ * @copyright Copyright (c) 2024 Vitor Mattos <[email protected]>
+ *
+ * @author Vitor Mattos <[email protected]>
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+namespace OCA\Libresign\Middleware;
+
+use OCA\Files\Controller\ViewController;
+use OCP\AppFramework\Controller;
+use OCP\AppFramework\Http\ContentSecurityPolicy;
+use OCP\AppFramework\Http\Response;
+use OCP\AppFramework\Middleware;
+
+class GlobalInjectionMiddleware extends Middleware {
+	public function afterController(Controller $controller, string $methodName, Response $response) {
+		if ($controller instanceof ViewController) {
+			$policy = new ContentSecurityPolicy();
+			$policy->allowEvalScript(true);
+			$response->setContentSecurityPolicy($policy);
+		}
+		return $response;
+	}
+}

src/Components/File/AppFilesTab.vue

@@ -24,7 +24,6 @@ export default {
 		return {
 			file: {},
 			signers: [],
-			name: '',
 			requestedBy: {},
 			requestDate: '',
 		}
@@ -49,14 +48,13 @@ export default {
 			this.signers = []
 			this.file = {
 				nodeId: fileInfo.id,
+				name: fileInfo.name
 			}
-			this.name = fileInfo.name
 			this.requestedBy = {}
 			this.requestDate = ''
 			try {
 				const response = await axios.get(generateOcsUrl(`/apps/libresign/api/v1/file/validate/file_id/${fileInfo.id}`))
 				this.signers = response.data.signers
-				this.name = response.data.name
 				this.file.uuid = response.data.uuid
 				this.requestedBy = response.data.requested_by
 				this.requestDate = response.data.request_date

src/Components/Request/RequestSignature.vue

@@ -168,7 +168,7 @@ export default {
 		},
 		async save() {
 			const params = {
-				name: this.name,
+				name: this.name.length > 0 ? this.name : this.file?.name,
 				users: [],
 			}
 			this.dataSigners.forEach(signer => {

tests/psalm-baseline.xml

@@ -72,6 +72,11 @@
       <code>LoadSidebar</code>
     </UndefinedClass>
   </file>
+  <file src="lib/Middleware/GlobalInjectionMiddleware.php">
+    <UndefinedClass occurrences="1">
+      <code>ViewController</code>
+    </UndefinedClass>
+  </file>
   <file src="lib/Migration/Version2040Date20211027183759.php">
     <MissingDependency occurrences="3">
       <code>$this-&gt;root</code>