Skip to content

fix(webdav): support blank-password nextcloud shares#133

Merged
Orinks merged 1 commit into
devfrom
codex/webdav-blank-password-nextcloud
May 26, 2026
Merged

fix(webdav): support blank-password nextcloud shares#133
Orinks merged 1 commit into
devfrom
codex/webdav-blank-password-nextcloud

Conversation

@Orinks
Copy link
Copy Markdown
Owner

@Orinks Orinks commented May 26, 2026

Summary

  • Force WebDAV Basic auth through the requests session when a username is provided, including blank-password shares.
  • Treat trailing-slash WebDAV fallback paths as directories when SabreDAV/Nextcloud omits explicit directory metadata from info responses.
  • Add regression coverage for blank-password WebDAV auth and sparse directory info responses.

Test Plan

  • uv run python -m ruff check src\portkeydrop\protocols.py tests\test_protocols.py
  • uv run python -m ruff format --check src\portkeydrop\protocols.py tests\test_protocols.py
  • uv run python -m pytest tests/test_protocols.py -k webdav
  • Live WebDAV probe against the reported Nextcloud public share with blank password and three subdirectories
  • uv run python -m pytest

@Orinks Orinks force-pushed the codex/webdav-blank-password-nextcloud branch from 48447ea to 3beefda Compare May 26, 2026 17:46
@Orinks Orinks merged commit 1555248 into dev May 26, 2026
6 checks passed
@Orinks Orinks deleted the codex/webdav-blank-password-nextcloud branch May 26, 2026 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant