fix: Fix one more Svace defect (#492)

* fix: Fix one more Svace defect Signed-off-by: Oleg Kopysov <o.kopysov@samsung.com> * fix: Added error messages with more details Signed-off-by: Oleg Kopysov <o.kopysov@samsung.com> --------- Signed-off-by: Oleg Kopysov <o.kopysov@samsung.com>
Samsung · Apr 10, 2024 · 2155207 · 2155207
1 parent 53193bf
commit 2155207
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/pom.xml b/pom.xml
@@ -155,6 +155,7 @@
                             <goal>single</goal>
                         </goals>
                         <configuration>
+                            <appendAssemblyId>true</appendAssemblyId>
                             <descriptorRefs>
                                 <descriptorRef>jar-with-dependencies</descriptorRef>
                             </descriptorRefs>
@@ -164,7 +165,6 @@
                                     <mainClass>com.lpvs.LicensePreValidationService</mainClass>
                                 </manifest>
                             </archive>
-                            <appendAssemblyId>false</appendAssemblyId>
                         </configuration>
                     </execution>
                 </executions>

diff --git a/src/main/java/com/lpvs/entity/LPVSFile.java b/src/main/java/com/lpvs/entity/LPVSFile.java
@@ -156,7 +156,7 @@ public String convertBytesToLinesNumbers() {
 
         if (matchedLines == null || matchedLines.isEmpty()) {
             return "";
-        } else if (matchedLines != null && !matchedLines.startsWith("BYTES:")) {
+        } else if (!matchedLines.startsWith("BYTES:")) {
             return matchedLines;
         }
 

diff --git a/src/main/java/com/lpvs/service/scan/scanner/LPVSScanossDetectService.java b/src/main/java/com/lpvs/service/scan/scanner/LPVSScanossDetectService.java
@@ -281,7 +281,8 @@ public List<LPVSFile> checkLicenses(LPVSQueue webhookConfig) {
             // close reader
             reader.close();
         } catch (IOException | IllegalArgumentException ex) {
-            log.error(ex.toString());
+            log.error("Error while processing Webhook ID = " + webhookConfig.getId());
+            log.error(ex.getMessage());
         }
         return detectedFiles;
     }