Skip to content

Comments

refactor(services/webdav): split webdav service into separate crate#7065

Merged
Xuanwo merged 1 commit intoapache:mainfrom
PiyushXCoder:webdav
Dec 19, 2025
Merged

refactor(services/webdav): split webdav service into separate crate#7065
Xuanwo merged 1 commit intoapache:mainfrom
PiyushXCoder:webdav

Conversation

@PiyushXCoder
Copy link
Contributor

@PiyushXCoder PiyushXCoder commented Dec 19, 2025

Which issue does this PR close?

Closes #6930

Rationale for this change

Relates to #6829

What changes are included in this PR?

This PR extracts the webdav service implementation from opendal-core into a standalone crate opendal-service-webdav, while keeping the public
surface opendal::services::Webdav unchanged.

  • Keeps feature flag services-webdav intact.
  • Moves webdav-specific dependencies (anyhow, mea, quick-xml with serialization features) to the new service crate.
  • Removed the old in-tree webdav service module + its feature/deps from opendal-core.

Are there any user-facing changes?

No. The public API opendal::services::Webdav remains unchanged and accessible via the services-webdav feature flag.

AI Usage Statement

None AI was used for the task

@PiyushXCoder PiyushXCoder requested a review from Xuanwo as a code owner December 19, 2025 09:30
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" labels Dec 19, 2025
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for working on this!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 19, 2025
@Xuanwo Xuanwo merged commit 66f08c7 into apache:main Dec 19, 2025
336 checks passed
@PiyushXCoder PiyushXCoder deleted the webdav branch December 19, 2025 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[core split] Service: webdav

2 participants