Upgrade tus-js-client

[Acconut]

Initial checklist

• I understand this is a bug report and questions should be posted in the Community Forum
• I searched issues and couldn’t find anything (or linked relevant results below)

Link to runnable example

No response

Steps to reproduce

The tus plugin and companion currently require tus-js-client@^3.0.0, which seems to be resolved to v3.0.1 when installed. However, tus-js-client v3.1.1 included an important fix to prevent tus-js-client from being stuck in a loop sending empty PATCH requests (see tus/tus-js-client#606). v3.1.2 includes a fix for progress events in Node.js, which was reported to me by you (see tus/tus-js-client#656).

Expected behavior

A newer version should be installed.

Actual behavior

An older version is installed.

[mifi]

not sure if there's anything we need to do, because tus-js-client@^3.0.0 semver range resolves to 3.1.2 too

[Acconut]

That's also true. I just checked and if you install companion via npm or yarn you will get a recent tus-js-client. I am just wondering why we had an old tus-js-client version in use in the api2 repo.

[mifi]

because yarn.lock file is designed to prevent packages from being updated without explicitly updating them

[Acconut]

That makes sense, thanks for helping me understand this. I still think it would not hurt to update tus-js-client in Uppy, even though it is not absolutely necessary. But I will leave that for you to decide :) So feel free to close this if you think there is nothing to do.