Merge pull request #15 from JSignPdf/bugfix/redirect-strerr-to-stdout

jeidison · web-flow · commit abd3a2ef22d2 · 2022-11-14T09:00:12.000-03:00
Redirect stderr to stdout and prevent display error
diff --git a/src/Sign/JSignService.php b/src/Sign/JSignService.php
@@ -60,7 +60,7 @@ public function getVersion(JSignParam $params)
         }
         $this->throwIf(!file_exists($jSignPdf), 'Jar of JSignPDF not found on path: '. $jSignPdf);
 
-        $command = "$java -jar $jSignPdf --version";
+        $command = "$java -jar $jSignPdf --version 2>&1";
         \exec($command, $output);
         if (empty($output) || strpos($output[0], 'version') === false) {
             return '';

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ public function getVersion(JSignParam $params)`
`60`	`60`	`}`
`61`	`61`	`$this->throwIf(!file_exists($jSignPdf), 'Jar of JSignPDF not found on path: '. $jSignPdf);`
`62`	`62`
`63`		`- $command = "$java -jar $jSignPdf --version";`
	`63`	`+ $command = "$java -jar $jSignPdf --version 2>&1";`
`64`	`64`	`\exec($command, $output);`
`65`	`65`	`if (empty($output) \|\| strpos($output[0], 'version') === false) {`
`66`	`66`	`return '';`