Browse code

Cleanup - no message was set in 2 cases for S3UploadError

Florent Viard authored on 2020/04/14 08:11:12
Showing 1 changed files
... ...
@@ -1649,8 +1649,9 @@ class S3(object):
1649 1649
                     return self.send_file(request, stream, labels, buffer, throttle,
1650 1650
                                           retries - 1, offset, chunk_size, use_expect_continue)
1651 1651
                 else:
1652
-                    warning("Too many failures. Giving up on '%s'" % (filename))
1653
-                    raise S3UploadError
1652
+                    warning("Too many failures. Giving up on '%s'" % filename)
1653
+                    raise S3UploadError("Too many failures. Giving up on '%s'"
1654
+                                        % filename)
1654 1655
 
1655 1656
             ## Non-recoverable error
1656 1657
             raise S3Error(response)
... ...
@@ -1666,7 +1667,8 @@ class S3(object):
1666 1666
                                       retries - 1, offset, chunk_size, use_expect_continue)
1667 1667
             else:
1668 1668
                 warning("Too many failures. Giving up on '%s'" % (filename))
1669
-                raise S3UploadError
1669
+                raise S3UploadError("Too many failures. Giving up on '%s'"
1670
+                                    % filename)
1670 1671
 
1671 1672
         return response
1672 1673