Refactor logging messages and remove redundant debug statements

Author: lennartdohmann

.devcontainer/devcontainer.json

@@ -12,10 +12,10 @@
         "jetmartin.bats",
         "ms-azuretools.vscode-containers",
         "github.vscode-github-actions",
-        "ms-vscode.makefile-tools",
         "ms-vscode-remote.remote-containers",
         "DEVSENSE.phptools-vscode",
-        "redhat.vscode-yaml"
+        "redhat.vscode-yaml",
+        "timonwong.shellcheck"
       ]
     }
   },

lib/Command/TagUnscannedCommand.php

@@ -42,7 +42,7 @@ protected function configure(): void {
 	#[\Override]
 	protected function execute(InputInterface $input, OutputInterface $output): int {
 		$logger = new ConsoleCommandLogger($this->logger, $output);
-		$logger->info('taggings files as unscanned');
+		$logger->info('tagging files as unscanned');
 		$start = microtime(true);
 		$taggedFilesCount = $this->tagUnscannedService
 			->withLogger($logger)

lib/Service/VerdictService.php

@@ -80,12 +80,6 @@ public function __construct(
 	public function scanFileById(int $fileId): VaasVerdict {
 		$node = $this->fileService->getNodeFromFileId($fileId);
 		$filePath = $node->getStorage()->getLocalFile($node->getInternalPath());
-		if (file_exists($filePath) === false) {
-			$this->logger->debug('Could not Scan File. File does not exist: ' . $filePath);
-		}
-		if (is_dir($filePath)) {
-			$this->logger->debug('Could not Scan File. File is a directory: ' . $filePath);
-		}
 		if (self::isFileTooLargeToScan($filePath)) {
 			$this->tagService->setTag($fileId, TagService::WONT_SCAN, silent: true);
 			throw new EntityTooLargeException('File is too large');
@@ -127,7 +121,6 @@ public function isFileTooLargeToScan(string $path): bool {
 	 */
 	public function isAllowedToScan(string $filePath): bool {
 		$doNotScanThis = $this->getDoNotScanThis();
-		$this->logger->debug('doNotScanThis: ' . implode(', ', $doNotScanThis));
 		foreach ($doNotScanThis as $doNotScanThisItem) {
 			if (str_contains(strtolower($filePath), strtolower($doNotScanThisItem))) {
 				return false;
@@ -137,7 +130,6 @@ public function isAllowedToScan(string $filePath): bool {
 		if (count($scanOnlyThis) === 0) {
 			return true;
 		}
-		$this->logger->debug('scanOnlyThis: ' . implode(', ', $scanOnlyThis));
 		foreach ($scanOnlyThis as $scanOnlyThisItem) {
 			if (str_contains(strtolower($filePath), strtolower($scanOnlyThisItem))) {
 				return true;