Skip to content

[BR-2340]: fix/webdav lookup failure status - #679

Open
jzunigax2 wants to merge 2 commits into
mainfrom
fix/webdav-lookup-failure-status
Open

jzunigax2 wants to merge 2 commits into
mainfrom
fix/webdav-lookup-failure-status

Conversation

@jzunigax2

Copy link
Copy Markdown
Contributor

A Drive API path lookup that timed out or failed transcitiently was reported to WebDAV clients as 404 Not Found. QNAP HBS3 read a this backend timeout as "Target path does not exist" and acted on it: re-creating folders, re-uploading files, skipping overwrites. Checking logs I could verify POST /folders that would fail as the folder already existed.

So lookup failures are now classified rather than all collapsed under 404

A Drive API lookup that timed out or failed was reported to WebDAV clients
as a 404, so QNAP HBS3 read a transient backend timeout as "Target path
does not exist" and went on to re-create folders, re-upload files and skip
overwrites.

Lookup failures are now classified: only a real 404 or a non-EXISTS item
means absent. Timeouts, 408/429/5xx and network failures return 503 with
Retry-After, and a rejected session returns 502 rather than a retry loop.
The cached uuid is only evicted on a confirmed 404, and the metadata
mappers reject a response body they cannot map.
@jzunigax2 jzunigax2 self-assigned this Sep 18, 2026
@jzunigax2 jzunigax2 added the bug Something isn't working label Sep 18, 2026
@jzunigax2
jzunigax2 force-pushed the fix/webdav-lookup-failure-status branch from 0e3adc0 to 1806add Compare September 18, 2026 20:06
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant