hnakamor opened a new issue, #13166:
URL: https://github.com/apache/druid/issues/13166

   ### Affected Version
   
   Druid 24.0.0
   
   ### Description
   
   I noticed that s3-extensions uses the getETag() value directly in 
MatchingETagConstraint().
   
https://github.com/apache/druid/blob/275f834b2a9f23cec59dfeeb9ab44ac9081620fe/extensions-core/s3-extensions/src/main/java/org/apache/druid/storage/s3/S3TaskLogs.java#L98
   
   Since getEtag() returns the value with leading and trailing quotes removed, 
using this value as is will result in an error response of "HTTP/1.1 412 
Precondition Failed" in some s3 applications.
   https://github.com/minio/minio-go/issues/687
   
   In environments where this error is returned, you will not be able to view 
Druid task reports or task logs from the Overlord API or the web console.
   
   Why not follow RFC7232 and add leading and trailing quotation marks?
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to