> @@ -101,9 +102,9 @@ public void endElement(String uri, String name, String 
> qName) {
>        } else if (qName.equals("ETag")) {
>           String currentETag = currentOrNull(currentText);
>           builder.eTag(currentETag);
> -         currentETag = currentETag.replace("\"", "");
> -         if (!MULTIPART_BLOB_ETAG.matcher(currentETag).matches()) {
> -            builder.contentMD5(base16().lowerCase().decode(currentETag));
> +         Matcher matcher = ETAG_CONTENT_MD5_PATTERN.matcher(currentETag);
> +         if (matcher.matches()) {

We're inverting the logic here (was `!...matches()`) - is that intentional? 
Also dropping support for etags of the form "AAAA-1111"...is that OK?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/493/files#r16429417

Reply via email to