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
Currently, tus-js-client supports onShouldRetryhttps://github.com/tus/tus-js-client/blob/master/docs/api.md#onshouldretry, which is also used internally by the @uppy/tus plugin. I'm currently having an issue when I'd like to also retry 401 responses after I refetch a new, valid auth token using a combination of onBeforeRequest and onAfterResponse.
Currently, I'm extending the plugin and replacing the bits I need to achieve this (upload method). But this would feel like a pretty good addition to the @uppy/tus.
The text was updated successfully, but these errors were encountered:
The retrying with exponential backoff is intricate and error prone logic to write yourself. We write that for you and test it so you don't have to worry about it. Perhaps it's better to only expose what HTTP status codes you want to retry on?
Hello 👋
Currently, tus-js-client supports
onShouldRetry
https://github.com/tus/tus-js-client/blob/master/docs/api.md#onshouldretry, which is also used internally by the@uppy/tus
plugin. I'm currently having an issue when I'd like to also retry 401 responses after I refetch a new, valid auth token using a combination ofonBeforeRequest
andonAfterResponse
.Currently, I'm extending the plugin and replacing the bits I need to achieve this (upload method). But this would feel like a pretty good addition to the
@uppy/tus
.The text was updated successfully, but these errors were encountered: