Clean up signer

gaul · Oct 27, 2014 · 559b1d3 · 559b1d3
1 parent 5742323
commit 559b1d3
Showing 1 changed file with 8 additions and 11 deletions.
diff --git a/src/main/java/org/gaul/s3proxy/S3ProxyHandler.java b/src/main/java/org/gaul/s3proxy/S3ProxyHandler.java
@@ -975,17 +975,14 @@ private static String createAuthorizationSignature(
 
         // build string to sign
         StringBuilder builder = new StringBuilder()
-                .append(request.getMethod()).append('\n');
-        String contentMD5 = request.getHeader(HttpHeaders.CONTENT_MD5);
-        if (contentMD5 != null) {
-            builder.append(contentMD5);
-        }
-        builder.append('\n');
-        String contentType = request.getHeader(HttpHeaders.CONTENT_TYPE);
-        if (contentType != null) {
-            builder.append(contentType);
-        }
-        builder.append('\n');
+                .append(request.getMethod())
+                .append('\n')
+                .append(Strings.nullToEmpty(request.getHeader(
+                        HttpHeaders.CONTENT_MD5)))
+                .append('\n')
+                .append(Strings.nullToEmpty(request.getHeader(
+                        HttpHeaders.CONTENT_TYPE)))
+                .append('\n');
         String expires = request.getParameter("Expires");
         if (expires != null) {
             builder.append(expires);