Fix new checkstyle issues in distribution 54/122054/1
authorJim Hahn <jrh3@att.com>
Fri, 18 Jun 2021 18:31:38 +0000 (14:31 -0400)
committerJim Hahn <jrh3@att.com>
Fri, 18 Jun 2021 18:32:18 +0000 (14:32 -0400)
The new version of checkstyle identified new issues.  Fixed those.

Issue-ID: POLICY-3286
Change-Id: I15adece9fa2e136108febc0d74af6c0d55f975c5
Signed-off-by: Jim Hahn <jrh3@att.com>
plugins/reception-plugins/pom.xml
plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicy.java

index 94efaa3..cd2fc35 100644 (file)
@@ -2,6 +2,7 @@
   ============LICENSE_START=======================================================
    Copyright (C) 2018 Ericsson. All rights reserved.
    Modifications copyright (C) 2020 Bell Canada. All rights reserved.
+   Modifications Copyright (C) 2021 AT&T Intellectual Property. All rights reserved.
   ================================================================================
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -39,7 +40,6 @@
         <dependency>
             <groupId>org.onap.sdc.sdc-distribution-client</groupId>
             <artifactId>sdc-distribution-client</artifactId>
-            <version>1.4.1</version>
             <exclusions>
                 <exclusion>
                     <groupId>org.slf4j</groupId>
index af30c1f..fe7f2dc 100644 (file)
@@ -101,8 +101,9 @@ public class PolicyDecoderFileInCsarToPolicy implements PolicyDecoder<Csar, Tosc
      * ensures validation of entries in the Zipfile. Attempts to solve path
      * injection java security issues.
      *
-     * @param entry the ZipEntry to check
+     * @param entryName name of the ZipEntry to check
      * @param csarPath Absolute path to the csar the ZipEntry is in
+     * @param entrySize size of the ZipEntry
      * @return true if no injection detected, and it is a policy type  or policy file.
      * @throws PolicyDecodingException if the file size is too large
      */