Xuanwo commented on PR #6255: URL: https://github.com/apache/opendal/pull/6255#issuecomment-2933758190
> 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. Nice catch! I haven’t implemented that for `presign` yet, as it’s a bit more complex. Feel free to take it on if you’re interested! -- 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]
