From 7bbd3bbad1db75a4639a4233d946e860e20e45ed Mon Sep 17 00:00:00 2001 From: Michael Robinson Date: Thu, 20 Jun 2019 17:00:22 +0100 Subject: [PATCH 1/2] This is actually not required, I misunderstood how it was working. We just need to use --fail --- aws | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/aws b/aws index 7a7907d..a4057d2 100755 --- a/aws +++ b/aws @@ -1970,7 +1970,7 @@ if (!$cmd_data) if ($r !~ /^<\?xml/) { print $r; - exit 255; + exit; } $r =~ s/<\?xml.*?>\r?\s*//; $result .= $r; From c901d93504be28e54240809ab1e8cab81c404491 Mon Sep 17 00:00:00 2001 From: Michael Robinson Date: Thu, 20 Jun 2019 17:40:53 +0100 Subject: [PATCH 2/2] Remove Content-MD5: from headers we don't actually need this anymore, we get the ETag for free from AWS. Also it breaks v4 signing. --- aws | 1 - 1 file changed, 1 deletion(-) diff --git a/aws b/aws index a4057d2..b56de0b 100755 --- a/aws +++ b/aws @@ -3031,7 +3031,6 @@ sub s3 $md5 = encode_base64(pack("H*", (split(" ", qx[md5sum @{[cq($file)]}]))[0]), ""); } - push @header, "Content-MD5: $md5"; print STDERR "setting $header[$#header]\n" if $v; } }