QuakeWang opened a new pull request, #7888: URL: https://github.com/apache/opendal/pull/7888
# Which issue does this PR close? Closes #. # Rationale for this change COS and TOS used decoded version IDs when building `versionId` query parameters for read/stat/delete requests. If a version ID contains query-reserved characters like `&`, `=`, `+`, or `%`, the generated URI can be parsed as different query parameters instead of a single encoded `versionId`. # What changes are included in this PR? - Encode COS and TOS `versionId` query values before appending them to request URIs. - Preserve encoded COS query parameters after reqsign rebuilds signed URIs. - Extract delete request builders for COS/TOS so delete URI construction can be covered directly. - Add tests for read/stat/delete version ID URI encoding, including COS signed and presigned paths. # Are there any user-facing changes? Yes. COS and TOS read/stat/delete operations with version IDs containing query-reserved characters now build valid request URIs. # AI Usage Statement I used OpenAI Codex to implement the test in this change. -- 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]
