amunra commented on PR #3478: URL: https://github.com/apache/incubator-opendal/pull/3478#issuecomment-1792999114
> > I believe s3s-fs's behaviour is correct, given that the AWS S3 request/response docs state that the Etag field is optional > > I don't think this statement is correct since `Key` is this doc is also `Required: no`. And we know that it's impossible. > > I'm ok with this patch to allow responses that not contain `ETag`. But I want to let you know that all s3 compatible services return `ETag`. Fair :-) For what it's worth, the official AWS `boto3` Python client is flexible about this too and doesn't require the `ETag` either: Thank you for accepting this change! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
