fix: Properly handle errors in Upload Api (related to StorageError)

[hmpthz]

What kind of change does this PR introduce?

Fix the error handling of upload, update and uploadToSignedUrl APIs so that they properly return StorageError instead of API's error schema.

What is the current behavior?

Multiple issues and PRs have been discussing this error for over a year. #214 #165

There are a bunch of helper functions at lib/fetch.ts that catch errors returned by the server and construct StorageError instance. Nearly all requests are using these helpers, except for uploadOrUpdate and uploadToSignedUrl in StorageFileApi, making their return types inconsistent:

Typescript declarations say the error returns are like:

{
  data: null,
  error: StorageError
}

However, without the helper functions, they actually return the error schema:

{
  data: null,
  error: { error: string, message: string, statusCode: string }
}

What is the new behavior?

This fix replaces raw fetch for uploadOrUpdate and uploadToSignedUrl with post and put helper functions so that their returns match the type declarations.

Additional context

It appears upload APIs are not using helpers because _getRequestParams will force body to be json type, which is an easy fix.

But if you think this part should not be touched, an alternative solution could be just patch the typings.

{
  data: null,
  error: StorageError | ErrorSchema
}