Skip to content

fix: enhance error handling for network and server errors in retry logic#348

Open
egalvis27 wants to merge 1 commit into
feat/go-fuse-daemonfrom
feat/add-network-error-to-retry-handler
Open

fix: enhance error handling for network and server errors in retry logic#348
egalvis27 wants to merge 1 commit into
feat/go-fuse-daemonfrom
feat/add-network-error-to-retry-handler

Conversation

@egalvis27
Copy link
Copy Markdown

What is Changed / Added

  • Updated Drive client error mapping to classify transport-level failures without an HTTP response/status as NETWORK_ERROR.
  • Updated folder creation error mapping so NETWORK_ERROR is treated as transient and routed through the same retry path as server-side transient failures.
  • Validated the change with focused tests for interceptor/client behavior and folder creation flow.

Why

  • When copying folders from the file manager, mkdir could fail on the first attempt and then succeed on manual retry.
  • Logs showed the underlying cause was a transport timeout (for example ETIMEDOUT) on POST /folders, meaning a network failure with no HTTP status.
  • Previously, this scenario was not handled as a transient retryable error in the folder creation flow, so it surfaced as a final user-facing failure.
  • With this change, status-less network failures are retried automatically, reducing intermittent folder creation/upload failures and improving user experience.

@sonarqubecloud
Copy link
Copy Markdown

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