Merge commit 'refs/pull/284/head' of https://github.com/camptocamp/od…

…oo-cloud-platform into 13.0-2446

attachment_s3/models/ir_attachment.py

@@ -167,6 +167,15 @@ def _store_file_write(self, key, bin_data):
                 file.write(bin_data)
                 file.seek(0)
                 filename = 's3://%s/%s' % (bucket.name, key)
+                # if the object already exists skip re-uploading so that:
+                # we dont break retencion policy and we dont need update rights
+                # TODO handle this on a better way
+                try:
+                    if obj.content_length:
+                        _logger.info('Skip uploading object %s, already exists', filename)
+                        return filename
+                except Exception:
+                    pass
                 try:
                     obj.upload_fileobj(file)
                 except ClientError as error: