[I] bug: improper error handling for S3 copy operations [opendal]

["dentiny (via GitHub)" <gi...@apache.org>]

dentiny opened a new issue, #7293:
URL: https://github.com/apache/opendal/issues/7293

   ### Describe the bug
   
   Hi team, I think in the current implementation, we simply consider copy operation successful when we receive a 200 OK HTTP status: https://github.com/apache/opendal/blob/51d61b503612ca9c7b1c32c9f4556a6752da3d85/core/services/s3/src/backend.rs#L1085
   but according to the [doc](https://docs.aws.amazon.com/AmazonS3/latest/API/API_CopyObject.html), "A 200 OK response can contain either a success or an error".  So we need to further parse the response and decide whether the copy operation succeeds or not.
   
   ### Steps to Reproduce
   
   N/A
   
   ### Expected Behavior
   
   We should check and parse response for copy request even when HTTP status code is 200 OK.
   
   ### Additional Context
   
   _No response_
   
   ### Are you willing to submit a PR to fix this bug?
   
   - [x] Yes, I would like to submit a PR.


-- 
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: commits-unsubscribe@opendal.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] bug: improper error handling for S3 copy operations [opendal]

["Xuanwo (via GitHub)" <gi...@apache.org>]

Xuanwo closed issue #7293: bug: improper error handling for S3 copy operations
URL: https://github.com/apache/opendal/issues/7293


-- 
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: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org