You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by sc...@apache.org on 2018/11/25 15:29:35 UTC

svn commit: r1847416 - /tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java

Author: schultz
Date: Sun Nov 25 15:29:35 2018
New Revision: 1847416

URL: http://svn.apache.org/viewvc?rev=1847416&view=rev
Log:
Simplify.

Modified:
    tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java

Modified: tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java?rev=1847416&r1=1847415&r2=1847416&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java (original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java Sun Nov 25 15:29:35 2018
@@ -344,46 +344,46 @@ public class EncryptInterceptor extends
         /**
          * The fully-specified algorithm e.g. AES/CBC/PKCS5Padding.
          */
-        private String algorithm;
+        private final String algorithm;
 
         /**
          * The size of the initialization vector to use for encryption. This is
          * often, but not always, the same as the block size.
          */
-        private int ivSize;
+        private final int ivSize;
 
         /**
          * The cryptographic provider name.
          */
-        private String providerName;
+        private final String providerName;
 
         /**
          * The secret key to use for encryption and decryption operations.
          */
-        private SecretKeySpec secretKey;
+        private final SecretKeySpec secretKey;
 
         /**
          * A pool of Cipher objects. Ciphers are expensive to create, but not
          * to re-initialize, so we use a pool of them which grows as necessary.
          */
-        private ConcurrentLinkedQueue<Cipher> cipherPool;
+        private final ConcurrentLinkedQueue<Cipher> cipherPool;
 
         /**
          * A pool of SecureRandom objects. Each encrypt operation requires access
          * to a source of randomness. SecureRandom is thread-safe, but sharing a
          * single instance will likely be a bottleneck.
          */
-        private ConcurrentLinkedQueue<SecureRandom> randomPool;
+        private final ConcurrentLinkedQueue<SecureRandom> randomPool;
 
         public EncryptionManager(String algorithm, SecretKeySpec secretKey, String providerName)
             throws NoSuchAlgorithmException, NoSuchPaddingException, NoSuchProviderException {
-            setAlgorithm(algorithm);
-            setProviderName(providerName);
-            setSecretKey(secretKey);
+            this.algorithm = algorithm;
+            this.providerName = providerName;
+            this.secretKey = secretKey;
 
             cipherPool = new ConcurrentLinkedQueue<>();
             Cipher cipher = createCipher();
-            setIVSize(cipher.getBlockSize());
+            ivSize = cipher.getBlockSize();
             cipherPool.offer(cipher);
             randomPool = new ConcurrentLinkedQueue<>();
         }
@@ -394,34 +394,18 @@ public class EncryptInterceptor extends
             randomPool.clear();
         }
 
-        private void setAlgorithm(String algorithm) {
-            this.algorithm = algorithm;
-        }
-
         private String getAlgorithm() {
             return algorithm;
         }
 
-        private void setSecretKey(SecretKeySpec secretKey) {
-            this.secretKey = secretKey;
-        }
-
         private SecretKeySpec getSecretKey() {
             return secretKey;
         }
 
-        private void setIVSize(int size) {
-            ivSize = size;
-        }
-
         private int getIVSize() {
             return ivSize;
         }
 
-        private void setProviderName(String provider) {
-            providerName = provider;
-        }
-
         private String getProviderName() {
             return providerName;
         }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org