Repository navigation
Conversation
There was a problem hiding this comment.
Code Review
This pull request refactors HTTP/JSON resumable upload logic by introducing a shared ResumableUploadHeaders utility class to centralize header constants and parsing logic. It also updates ResumableUploadChunkCallable and ResumableUploadQueryStatusCallable to omit credentials and reset the EndpointContext for self-authenticating session URLs. Additionally, chunk granularity parsing has been integrated into the query status flow, and corresponding unit tests have been added. Feedback suggests avoiding the use of HttpJsonStatusCode.of when creating a StatusCode for internal exceptions to prevent incorrect classification of errors as recoverable, recommending an anonymous implementation of StatusCode instead.
|
|





parseChunkGranularity(now used in both start and query) into a newResumableUploadHeadersclass