geruh commented on PR #6255: URL: https://github.com/apache/opendal/pull/6255#issuecomment-2933728113
While working on this, I noticed that some of the options are specific to presign. I think I can add those in a follow-up, as this PR was getting large. However, it's worth noting that the current presign API doesn't follow the new xxx_option pattern and instead uses the with_ method chaining approach. Also, presign operations support the override* fields and a small subset of options (e.g., if_none_match). It may be worth investigating adding support for the options defined in #6213 to the presign operations. -- 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]
