You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
storage: cap consecutive routing redirects in bidi read #7200
In google-cloud-storage, the bidi write enforces a maximum limit on consecutive routing redirects (MAX_REDIRECTS_FOLLOWED = 3 in bidi_write::retry_redirect), aligning with other language SDKs.
However, in the bidi read (bidi::RetryRedirect and bidi::ResumeRedirect), redirect errors (Code::Aborted with BidiReadObjectRedirectedError) are converted to RetryResult::Continue, but there is currently no dedicated cap on consecutive redirects followed. Because redirect errors are classified separately from standard transient errors, attempt-limited retry policies do not deterministically bound consecutive redirect chains before the overall request timeout.
Expected Behavior
Consecutive redirects in bidi read should be bounded by a fixed limit (e.g. 3 consecutive redirects / 4 total attempts, matching bidi_write and other SDKs), returning an error if the limit is exceeded.
Activity
test(storage): align max redirect test with googleapis#7200