YuangGao opened a new pull request, #7637: URL: https://github.com/apache/opendal/pull/7637
# Which issue does this PR close? Part of #5486. # Rationale for this change The webdav service already supports read but ignored the `OpRead` argument, so conditional headers never reached the GET request. Per RFC 7232 (HTTP/1.1 conditional requests, inherited by WebDAV servers), `If-Match` / `If-None-Match` / `If-Modified-Since` / `If-Unmodified-Since` are standard on GET. # What changes are included in this PR? - Inject the four conditional headers in `webdav_get`. - Declare `read_with_if_match`, `read_with_if_none_match`, `read_with_if_modified_since`, `read_with_if_unmodified_since` on the webdav service capability. Stat-side conditional support (PROPFIND) is left for a follow-up because `webdav_stat` doesn't currently accept `OpStat` and PROPFIND conditional semantics are server-dependent. # Are there any user-facing changes? Yes — `op.reader_with(path).if_match(...)` (and the three siblings) now work against webdav backends. No breaking changes; # AI Usage Statement AI-assisted implementation. -- 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]
