Bläddra i källkod

dependency updates and spotbugs fixes

Jason Rivard 5 år sedan
förälder
incheckning
6aa32cebb6

+ 4 - 1
build/spotbugs-exclude.xml

@@ -21,7 +21,10 @@
 
 
 <FindBugsFilter>
 <FindBugsFilter>
     <Match>
     <Match>
-        <Bug pattern="SE_NO_SERIALVERSIONID,RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE,SQL_PREPARED_STATEMENT_GENERATED_FROM_NONCONSTANT_STRING"/>
+        <Bug pattern="RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE"/>
+    </Match>
+    <Match>
+        <Bug pattern="SQL_PREPARED_STATEMENT_GENERATED_FROM_NONCONSTANT_STRING"/>
     </Match>
     </Match>
     <Match>
     <Match>
         <!-- due to bug https://github.com/spotbugs/spotbugs/issues/493 in spotbugs 3.1.3 -->
         <!-- due to bug https://github.com/spotbugs/spotbugs/issues/493 in spotbugs 3.1.3 -->

+ 3 - 3
data-service/pom.xml

@@ -46,7 +46,7 @@
             <plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-war-plugin</artifactId>
                 <artifactId>maven-war-plugin</artifactId>
-                <version>3.2.2</version>
+                <version>3.2.3</version>
                 <configuration>
                 <configuration>
                     <archiveClasses>true</archiveClasses>
                     <archiveClasses>true</archiveClasses>
                     <packagingExcludes>WEB-INF/classes</packagingExcludes>
                     <packagingExcludes>WEB-INF/classes</packagingExcludes>
@@ -130,7 +130,7 @@
         <dependency>
         <dependency>
             <groupId>org.apache.commons</groupId>
             <groupId>org.apache.commons</groupId>
             <artifactId>commons-lang3</artifactId>
             <artifactId>commons-lang3</artifactId>
-            <version>3.9</version>
+            <version>3.10</version>
         </dependency>
         </dependency>
         <dependency>
         <dependency>
             <groupId>com.sun.mail</groupId>
             <groupId>com.sun.mail</groupId>
@@ -140,7 +140,7 @@
         <dependency>
         <dependency>
             <groupId>org.apache.httpcomponents</groupId>
             <groupId>org.apache.httpcomponents</groupId>
             <artifactId>httpclient</artifactId>
             <artifactId>httpclient</artifactId>
-            <version>4.5.11</version>
+            <version>4.5.12</version>
         </dependency>
         </dependency>
         <dependency>
         <dependency>
             <groupId>log4j</groupId>
             <groupId>log4j</groupId>

+ 1 - 1
onejar/pom.xml

@@ -16,7 +16,7 @@
     <name>PWM Password Self Service: Executable Server JAR</name>
     <name>PWM Password Self Service: Executable Server JAR</name>
 
 
     <properties>
     <properties>
-        <tomcat.version>9.0.31</tomcat.version>
+        <tomcat.version>9.0.34</tomcat.version>
     </properties>
     </properties>
 
 
     <build>
     <build>

+ 8 - 8
pom.xml

@@ -120,7 +120,7 @@
             <plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-javadoc-plugin</artifactId>
                 <artifactId>maven-javadoc-plugin</artifactId>
-                <version>3.1.1</version>
+                <version>3.2.0</version>
                 <executions>
                 <executions>
                     <execution>
                     <execution>
                         <goals>
                         <goals>
@@ -224,7 +224,7 @@
                     <dependency>
                     <dependency>
                         <groupId>com.puppycrawl.tools</groupId>
                         <groupId>com.puppycrawl.tools</groupId>
                         <artifactId>checkstyle</artifactId>
                         <artifactId>checkstyle</artifactId>
-                        <version>8.30</version>
+                        <version>8.31</version>
                     </dependency>
                     </dependency>
                 </dependencies>
                 </dependencies>
                 <executions>
                 <executions>
@@ -303,12 +303,12 @@
             <plugin>
             <plugin>
                 <groupId>com.github.spotbugs</groupId>
                 <groupId>com.github.spotbugs</groupId>
                 <artifactId>spotbugs-maven-plugin</artifactId>
                 <artifactId>spotbugs-maven-plugin</artifactId>
-                <version>3.1.12.2</version>
+                <version>4.0.0</version>
                 <dependencies>
                 <dependencies>
                     <dependency>
                     <dependency>
                         <groupId>com.github.spotbugs</groupId>
                         <groupId>com.github.spotbugs</groupId>
                         <artifactId>spotbugs</artifactId>
                         <artifactId>spotbugs</artifactId>
-                        <version>4.0.0</version>
+                        <version>4.0.1</version>
                     </dependency>
                     </dependency>
                 </dependencies>
                 </dependencies>
                 <configuration>
                 <configuration>
@@ -350,7 +350,7 @@
             <plugin> <!-- checks owsp vulnerability database -->
             <plugin> <!-- checks owsp vulnerability database -->
                 <groupId>org.owasp</groupId>
                 <groupId>org.owasp</groupId>
                 <artifactId>dependency-check-maven</artifactId>
                 <artifactId>dependency-check-maven</artifactId>
-                <version>5.3.0</version>
+                <version>5.3.2</version>
                 <executions>
                 <executions>
                     <execution>
                     <execution>
                         <goals>
                         <goals>
@@ -373,7 +373,7 @@
         <dependency>
         <dependency>
             <groupId>com.github.spotbugs</groupId>
             <groupId>com.github.spotbugs</groupId>
             <artifactId>spotbugs-annotations</artifactId>
             <artifactId>spotbugs-annotations</artifactId>
-            <version>4.0.0</version>
+            <version>4.0.1</version>
             <scope>provided</scope>
             <scope>provided</scope>
         </dependency>
         </dependency>
 
 
@@ -387,7 +387,7 @@
         <dependency>
         <dependency>
             <groupId>org.mockito</groupId>
             <groupId>org.mockito</groupId>
             <artifactId>mockito-core</artifactId>
             <artifactId>mockito-core</artifactId>
-            <version>3.3.0</version>
+            <version>3.3.3</version>
             <scope>test</scope>
             <scope>test</scope>
         </dependency>
         </dependency>
         <dependency>
         <dependency>
@@ -399,7 +399,7 @@
         <dependency>
         <dependency>
             <groupId>com.github.tomakehurst</groupId>
             <groupId>com.github.tomakehurst</groupId>
             <artifactId>wiremock</artifactId>
             <artifactId>wiremock</artifactId>
-            <version>2.26.1</version>
+            <version>2.26.3</version>
             <scope>test</scope>
             <scope>test</scope>
         </dependency>
         </dependency>
         <dependency>
         <dependency>

+ 1 - 1
pwm-cr/pom.xml

@@ -41,7 +41,7 @@
         <dependency>
         <dependency>
             <groupId>org.bouncycastle</groupId>
             <groupId>org.bouncycastle</groupId>
             <artifactId>bcpkix-jdk15on</artifactId>
             <artifactId>bcpkix-jdk15on</artifactId>
-            <version>1.64</version>
+            <version>1.65</version>
         </dependency>
         </dependency>
     </dependencies>
     </dependencies>
 
 

+ 6 - 6
server/pom.xml

@@ -202,7 +202,7 @@
         <dependency>
         <dependency>
             <groupId>org.apache.directory.api</groupId>
             <groupId>org.apache.directory.api</groupId>
             <artifactId>api-all</artifactId>
             <artifactId>api-all</artifactId>
-            <version>2.0.0</version>
+            <version>2.0.1</version>
         </dependency>
         </dependency>
         <dependency>
         <dependency>
             <groupId>commons-net</groupId>
             <groupId>commons-net</groupId>
@@ -227,7 +227,7 @@
         <dependency>
         <dependency>
             <groupId>org.apache.commons</groupId>
             <groupId>org.apache.commons</groupId>
             <artifactId>commons-lang3</artifactId>
             <artifactId>commons-lang3</artifactId>
-            <version>3.9</version>
+            <version>3.10</version>
         </dependency>
         </dependency>
         <dependency>
         <dependency>
             <groupId>commons-validator</groupId>
             <groupId>commons-validator</groupId>
@@ -242,7 +242,7 @@
         <dependency>
         <dependency>
             <groupId>org.apache.httpcomponents</groupId>
             <groupId>org.apache.httpcomponents</groupId>
             <artifactId>httpclient</artifactId>
             <artifactId>httpclient</artifactId>
-            <version>4.5.11</version>
+            <version>4.5.12</version>
         </dependency>
         </dependency>
         <dependency>
         <dependency>
             <groupId>org.graylog2</groupId>
             <groupId>org.graylog2</groupId>
@@ -267,12 +267,12 @@
         <dependency>
         <dependency>
             <groupId>org.bouncycastle</groupId>
             <groupId>org.bouncycastle</groupId>
             <artifactId>bcprov-jdk15on</artifactId>
             <artifactId>bcprov-jdk15on</artifactId>
-            <version>1.64</version>
+            <version>1.65</version>
         </dependency>
         </dependency>
         <dependency>
         <dependency>
             <groupId>org.bouncycastle</groupId>
             <groupId>org.bouncycastle</groupId>
             <artifactId>bcpkix-jdk15on</artifactId>
             <artifactId>bcpkix-jdk15on</artifactId>
-            <version>1.64</version>
+            <version>1.65</version>
         </dependency>
         </dependency>
         <dependency>
         <dependency>
             <groupId>jaxen</groupId>
             <groupId>jaxen</groupId>
@@ -302,7 +302,7 @@
         <dependency>
         <dependency>
             <groupId>com.blueconic</groupId>
             <groupId>com.blueconic</groupId>
             <artifactId>browscap-java</artifactId>
             <artifactId>browscap-java</artifactId>
-            <version>1.2.14</version>
+            <version>1.2.16</version>
         </dependency>
         </dependency>
         <dependency>
         <dependency>
             <groupId>org.jetbrains.xodus</groupId>
             <groupId>org.jetbrains.xodus</groupId>

+ 2 - 0
server/src/main/java/password/pwm/bean/UserIdentity.java

@@ -40,6 +40,8 @@ import java.util.StringTokenizer;
 
 
 public class UserIdentity implements Serializable, Comparable
 public class UserIdentity implements Serializable, Comparable
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private static final String CRYPO_HEADER = "ui_C-";
     private static final String CRYPO_HEADER = "ui_C-";
     private static final String DELIM_SEPARATOR = "|";
     private static final String DELIM_SEPARATOR = "|";
 
 

+ 1 - 0
server/src/main/java/password/pwm/config/profile/PwmPasswordPolicy.java

@@ -54,6 +54,7 @@ import java.util.regex.Pattern;
  */
  */
 public class PwmPasswordPolicy implements Profile, Serializable
 public class PwmPasswordPolicy implements Profile, Serializable
 {
 {
+    private static final long serialVersionUID = 1L;
 
 
     private static final PwmLogger LOGGER = PwmLogger.forClass( PwmPasswordPolicy.class );
     private static final PwmLogger LOGGER = PwmLogger.forClass( PwmPasswordPolicy.class );
 
 

+ 1 - 0
server/src/main/java/password/pwm/config/value/NamedSecretValue.java

@@ -52,6 +52,7 @@ import java.util.Optional;
 
 
 public class NamedSecretValue implements StoredValue
 public class NamedSecretValue implements StoredValue
 {
 {
+    private static final long serialVersionUID = 1L;
 
 
     private final transient LazySupplier<String> valueHashSupplier = new LazySupplier<>( () -> AbstractValue.valueHashComputer( NamedSecretValue.this ) );
     private final transient LazySupplier<String> valueHashSupplier = new LazySupplier<>( () -> AbstractValue.valueHashComputer( NamedSecretValue.this ) );
 
 

+ 2 - 0
server/src/main/java/password/pwm/config/value/PasswordValue.java

@@ -45,6 +45,8 @@ import java.util.Optional;
 
 
 public class PasswordValue implements StoredValue
 public class PasswordValue implements StoredValue
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private final transient LazySupplier<String> valueHashSupplier = new LazySupplier<>( () -> AbstractValue.valueHashComputer( PasswordValue.this ) );
     private final transient LazySupplier<String> valueHashSupplier = new LazySupplier<>( () -> AbstractValue.valueHashComputer( PasswordValue.this ) );
 
 
     private final PasswordData value;
     private final PasswordData value;

+ 2 - 0
server/src/main/java/password/pwm/http/ContextManager.java

@@ -77,6 +77,8 @@ import java.util.concurrent.atomic.AtomicInteger;
 
 
 public class ContextManager implements Serializable
 public class ContextManager implements Serializable
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private static final PwmLogger LOGGER = PwmLogger.forClass( ContextManager.class );
     private static final PwmLogger LOGGER = PwmLogger.forClass( ContextManager.class );
     private static final SessionLabel SESSION_LABEL = SessionLabel.CONTEXT_SESSION_LABEL;
     private static final SessionLabel SESSION_LABEL = SessionLabel.CONTEXT_SESSION_LABEL;
 
 

+ 2 - 0
server/src/main/java/password/pwm/http/PwmSession.java

@@ -58,6 +58,8 @@ import java.util.Map;
  */
  */
 public class PwmSession implements Serializable
 public class PwmSession implements Serializable
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private static final PwmLogger LOGGER = PwmLogger.forClass( PwmSession.class );
     private static final PwmLogger LOGGER = PwmLogger.forClass( PwmSession.class );
 
 
     private final transient PwmApplication pwmApplication;
     private final transient PwmApplication pwmApplication;

+ 2 - 0
server/src/main/java/password/pwm/http/bean/ConfigManagerBean.java

@@ -30,6 +30,8 @@ import java.util.Set;
 @Data
 @Data
 public class ConfigManagerBean extends PwmSessionBean
 public class ConfigManagerBean extends PwmSessionBean
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private transient StoredConfiguration storedConfiguration;
     private transient StoredConfiguration storedConfiguration;
     private boolean passwordVerified;
     private boolean passwordVerified;
     private boolean configUnlockedWarningShown;
     private boolean configUnlockedWarningShown;

+ 4 - 0
server/src/main/java/password/pwm/http/bean/ForgottenPasswordBean.java

@@ -49,6 +49,8 @@ import java.util.Set;
 @EqualsAndHashCode( callSuper = false )
 @EqualsAndHashCode( callSuper = false )
 public class ForgottenPasswordBean extends PwmSessionBean
 public class ForgottenPasswordBean extends PwmSessionBean
 {
 {
+    private static final long serialVersionUID = 1L;
+
     @SerializedName( "pr" )
     @SerializedName( "pr" )
     private String profile;
     private String profile;
 
 
@@ -82,6 +84,8 @@ public class ForgottenPasswordBean extends PwmSessionBean
     @Data
     @Data
     public static class Progress implements Serializable
     public static class Progress implements Serializable
     {
     {
+        private static final long serialVersionUID = 1L;
+
         @SerializedName( "s" )
         @SerializedName( "s" )
         private boolean tokenSent;
         private boolean tokenSent;
 
 

+ 2 - 0
server/src/main/java/password/pwm/http/servlet/admin/UserDebugDataBean.java

@@ -36,6 +36,8 @@ import java.util.Map;
 @Builder
 @Builder
 public class UserDebugDataBean implements Serializable
 public class UserDebugDataBean implements Serializable
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private transient UserInfo userInfo;
     private transient UserInfo userInfo;
 
 
     private final PublicUserInfoBean publicUserInfoBean;
     private final PublicUserInfoBean publicUserInfoBean;

+ 2 - 0
server/src/main/java/password/pwm/http/servlet/helpdesk/HelpdeskVerificationStateBean.java

@@ -49,6 +49,8 @@ import java.util.TreeMap;
 
 
 class HelpdeskVerificationStateBean implements Serializable
 class HelpdeskVerificationStateBean implements Serializable
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private static final PwmLogger LOGGER = PwmLogger.forClass( HelpdeskVerificationStateBean.class );
     private static final PwmLogger LOGGER = PwmLogger.forClass( HelpdeskVerificationStateBean.class );
     public static final String PARAMETER_VERIFICATION_STATE_KEY = "verificationState";
     public static final String PARAMETER_VERIFICATION_STATE_KEY = "verificationState";
 
 

+ 2 - 0
server/src/main/java/password/pwm/ldap/search/SearchConfiguration.java

@@ -39,6 +39,8 @@ import java.util.Map;
 @Builder( toBuilder = true )
 @Builder( toBuilder = true )
 public class SearchConfiguration implements Serializable
 public class SearchConfiguration implements Serializable
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private String filter;
     private String filter;
     private String ldapProfile;
     private String ldapProfile;
     private String username;
     private String username;

+ 2 - 0
server/src/main/java/password/pwm/svc/wordlist/WordlistConfiguration.java

@@ -47,6 +47,8 @@ import java.util.function.Supplier;
 @Builder( toBuilder = true )
 @Builder( toBuilder = true )
 public class WordlistConfiguration implements Serializable
 public class WordlistConfiguration implements Serializable
 {
 {
+    private static final long serialVersionUID = 1L;
+
     static final int STREAM_BUFFER_SIZE = 1_1024_1024;
     static final int STREAM_BUFFER_SIZE = 1_1024_1024;
     static final PwmHashAlgorithm HASH_ALGORITHM = PwmHashAlgorithm.SHA256;
     static final PwmHashAlgorithm HASH_ALGORITHM = PwmHashAlgorithm.SHA256;
 
 

+ 2 - 0
server/src/main/java/password/pwm/util/LDAPPermissionCalculator.java

@@ -57,6 +57,8 @@ import java.util.TreeMap;
 
 
 public class LDAPPermissionCalculator implements Serializable
 public class LDAPPermissionCalculator implements Serializable
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private static final PwmLogger LOGGER = PwmLogger.forClass( LDAPPermissionCalculator.class );
     private static final PwmLogger LOGGER = PwmLogger.forClass( LDAPPermissionCalculator.class );
 
 
     private final transient StoredConfiguration storedConfiguration;
     private final transient StoredConfiguration storedConfiguration;

+ 2 - 0
server/src/main/java/password/pwm/util/PasswordData.java

@@ -43,6 +43,8 @@ import java.util.Arrays;
  */
  */
 public class PasswordData implements Serializable
 public class PasswordData implements Serializable
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private static final PwmLogger LOGGER = PwmLogger.forClass( PasswordData.class );
     private static final PwmLogger LOGGER = PwmLogger.forClass( PasswordData.class );
 
 
     private final byte[] passwordData;
     private final byte[] passwordData;

+ 2 - 0
server/src/main/java/password/pwm/ws/server/RestAuthentication.java

@@ -32,6 +32,8 @@ import java.util.Set;
 @Value
 @Value
 public class RestAuthentication implements Serializable
 public class RestAuthentication implements Serializable
 {
 {
+    private static final long serialVersionUID = 1L;
+
     private RestAuthenticationType type;
     private RestAuthenticationType type;
     private String namedSecretName;
     private String namedSecretName;
     private UserIdentity ldapIdentity;
     private UserIdentity ldapIdentity;

+ 2 - 2
webapp/pom.xml

@@ -91,7 +91,7 @@
             <plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-war-plugin</artifactId>
                 <artifactId>maven-war-plugin</artifactId>
-                <version>3.2.2</version>
+                <version>3.2.3</version>
                 <configuration>
                 <configuration>
                     <archiveClasses>false</archiveClasses>
                     <archiveClasses>false</archiveClasses>
                     <packagingExcludes>**/*.jsp</packagingExcludes>
                     <packagingExcludes>**/*.jsp</packagingExcludes>
@@ -182,7 +182,7 @@
                 <!-- builds xml file of dependencies and licenses for use in about page -->
                 <!-- builds xml file of dependencies and licenses for use in about page -->
                 <groupId>com.github.jinnovations</groupId>
                 <groupId>com.github.jinnovations</groupId>
                 <artifactId>attribution-maven-plugin</artifactId>
                 <artifactId>attribution-maven-plugin</artifactId>
-                <version>0.9.7</version>
+                <version>0.9.8</version>
                 <executions>
                 <executions>
                     <execution>
                     <execution>
                         <goals>
                         <goals>