@uppy/tus: fix onBeforeRequest type

[dschmidt]

c.f.

uppy/packages/@uppy/tus/src/index.ts

Lines 243 to 271 in d6156bb

	let userProvidedPromise
	if (typeof onBeforeRequest === 'function') {
	userProvidedPromise = onBeforeRequest(req, file)
	}
	if (hasProperty(queuedRequest, 'shouldBeRequeued')) {
	if (!queuedRequest.shouldBeRequeued) return Promise.reject()
	// TODO: switch to `Promise.withResolvers` on the next major if available.
	let done: () => void
	// eslint-disable-next-line promise/param-names
	const p = new Promise<void>((res) => {
	done = res
	})
	queuedRequest = this.requests.run(() => {
	if (file.isPaused) {
	queuedRequest.abort()
	}
	done()
	return () => {}
	})
	// If the request has been requeued because it was rate limited by the
	// remote server, we want to wait for `RateLimitedQueue` to dispatch
	// the re-try request.
	// Therefore we create a promise that the queue will resolve when
	// enough time has elapsed to expect not to be rate-limited again.
	// This means we can hold the Tus retry here with a `Promise.all`,
	// together with the returned value of the user provided
	// `onBeforeRequest` option callback (in case it returns a promise).
	await Promise.all([p, userProvidedPromise])

Sonarcloud was complaining because we returned a Promise in our onBeforeRequest handler:

owncloud/web#12052 (comment)

[github-actions]

Diff output files

No diff

[dschmidt]

No idea, why we have a promise there at all - but the type seems wrong anyhow 😁

[Murderlon]

Thanks!