> @@ -51,7 +51,17 @@ public void endElement(String uri, String name, String 
> qName) {
>        if (qName.equals("ETag")) {
>           this.currentETag = currentOrNull(currentText);
>        } else if (qName.equals("LastModified")) {
> -         this.currentLastModified = 
> dateParser.iso8601DateParse(currentOrNull(currentText));
> +         try {
> +            this.currentLastModified = dateParser
> +                  .iso8601DateParse(currentOrNull(currentText));
> +         } catch (IllegalArgumentException orig) {
> +            try {
> +               this.currentLastModified = dateParser
> +                     .iso8601SecondsDateParse(currentOrNull(currentText));

Please rework this pull request with an `iso8601OptionalMilliSecondsDateParse` 
helper instead of duplicating exception handling throughout the code.  Also add 
a comment explaining why this workaround exists and for which vendor and 
operations.  Have you reported this deviation to the vendor yet?

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

Reply via email to