PrefectHQ / prefect-aws

Prefect integrations with AWS.

https://PrefectHQ.github.io/prefect-aws/

Apache License 2.0

85 stars 40 forks source link

`S3Bucket.copy_object`: Only resolve target path with self if `to_bucket` is not defined #384

Closed dominictarro closed 8 months ago

dominictarro commented 9 months ago

S3Bucket.copy_object resolves the target path using the source bucket, regardless of whether the object goes to the source bucket or elsewhere.

https://github.com/PrefectHQ/prefect-aws/blob/87755b2d7fd979d7975486abdd56f2e4d191e2f7/prefect_aws/s3.py#L1238

Expectation / Proposal

Resolve the target path with self when the object is being copied within the bucket
Resolve the target path with the to_bucket when the object is being copied to it and it is an S3Bucket

Traceback / Example

[x] I would like to help contribute a pull request to resolve this!