Drop dependency on commons-codec

Since we have updated to Java 8 we can use JDK facilities to perform
Base64 encoding. There's no need to keep commons-codec.

Author: elreydetodo

edgegrid-signer-core/pom.xml

@@ -18,10 +18,6 @@
             <groupId>ch.qos.logback</groupId>
             <artifactId>logback-classic</artifactId>
         </dependency>
-        <dependency>
-            <groupId>commons-codec</groupId>
-            <artifactId>commons-codec</artifactId>
-        </dependency>
         <dependency>
             <groupId>org.apache.commons</groupId>
             <artifactId>commons-lang3</artifactId>

edgegrid-signer-core/src/main/java/com/akamai/edgegrid/signer/EdgeGridV1Signer.java

@@ -23,6 +23,7 @@
 import java.security.NoSuchAlgorithmException;
 import java.text.SimpleDateFormat;
 import java.util.ArrayList;
+import java.util.Base64;
 import java.util.Date;
 import java.util.List;
 import java.util.Map;
@@ -34,7 +35,6 @@
 import javax.crypto.Mac;
 import javax.crypto.spec.SecretKeySpec;
 
-import org.apache.commons.codec.binary.Base64;
 import org.apache.commons.lang3.StringEscapeUtils;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.commons.lang3.Validate;
@@ -195,12 +195,12 @@ private String getSignature(Request request, ClientCredential credential, String
 
     private String signAndEncode(String stringToSign, String signingKey) throws RequestSigningException {
         byte[] signatureBytes = sign(stringToSign, signingKey);
-        return Base64.encodeBase64String(signatureBytes);
+        return Base64.getEncoder().encodeToString(signatureBytes);
     }
 
     private String getSigningKey(String timeStamp, String clientSecret) throws RequestSigningException {
         byte[] signingKeyBytes = sign(timeStamp, clientSecret);
-        return Base64.encodeBase64String(signingKeyBytes);
+        return Base64.getEncoder().encodeToString(signingKeyBytes);
     }
 
     private String getDataToSign(String canonicalizedRequest, String authData) {
@@ -315,15 +315,15 @@ private String getContentHash(String requestMethod, byte[] requestBody, int maxB
             lengthToHash = maxBodySize;
         } else {
             if (log.isTraceEnabled()) {
-                log.trace("Content (Base64): {}", Base64.encodeBase64String(requestBody));
+                log.trace("Content (Base64): {}", Base64.getEncoder().encodeToString(requestBody));
             }
         }
 
         byte[] digestBytes = getHash(requestBody, 0, lengthToHash);
-        log.debug("Content hash (Base64): {}", Base64.encodeBase64String(digestBytes));
+        log.debug("Content hash (Base64): {}", Base64.getEncoder().encodeToString(digestBytes));
 
         // (mgawinec) I removed support for non-retryable content, that used to reset the content for downstream handlers
-        return Base64.encodeBase64String(digestBytes);
+        return Base64.getEncoder().encodeToString(digestBytes);
     }
 
 }

pom.xml

@@ -84,11 +84,6 @@
                 <version>1.1.7</version>
                 <scope>test</scope>
             </dependency>
-            <dependency>
-                <groupId>commons-codec</groupId>
-                <artifactId>commons-codec</artifactId>
-                <version>1.10</version>
-            </dependency>
             <dependency>
                 <groupId>com.github.tomakehurst</groupId>
                 <artifactId>wiremock</artifactId>