Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tus withCredentials option missing in TypeScript definition #2518

Closed
szh opened this issue Sep 8, 2020 · 3 comments
Closed

Tus withCredentials option missing in TypeScript definition #2518

szh opened this issue Sep 8, 2020 · 3 comments
Assignees
Labels
Bug Tus Resumable file uploading vis tus.io and Uppy Tus plugin

Comments

@szh
Copy link
Contributor

szh commented Sep 8, 2020

The withCredentials option mentinoed in the docs here: https://uppy.io/docs/tus/#withCredentials-false appears to be missing from the typescript definition file: https://github.com/transloadit/uppy/blob/master/packages/@uppy/tus/types/index.d.ts

As it happens, I don't think this option does anything anymore, though I can't find any record of it being removed. Is there a way to have cookies passed on to the Tus server at this point?

@goto-bus-stop goto-bus-stop added Tus Resumable file uploading vis tus.io and Uppy Tus plugin and removed Triage labels Sep 21, 2020
@goto-bus-stop
Copy link
Contributor

goto-bus-stop commented Sep 21, 2020

I guess it was removed from tus-js-client in the v2 upgrade. That means we had an unintentional backwards compatibility breaking change. We can add it back in @uppy/tus by doing something like this: https://github.com/tus/tus-js-client/blob/446c6a309546730d95396abe6e9728a61d99ee60/docs/api.md#onbeforerequest

@goto-bus-stop goto-bus-stop self-assigned this Sep 21, 2020
szh pushed a commit to szh/uppy that referenced this issue Sep 21, 2020
@szh
Copy link
Contributor Author

szh commented Sep 21, 2020

@goto-bus-stop Thanks for the direction. I added a pull request, #2544

szh pushed a commit to szh/uppy that referenced this issue Sep 23, 2020
szh pushed a commit to szh/uppy that referenced this issue Sep 23, 2020
@bedgerotto
Copy link
Contributor

Hi there, I submitted this PR (#2611 ) because I need to customize the onBeforeRequest callback inside the host application and I think this is related with thsi issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Tus Resumable file uploading vis tus.io and Uppy Tus plugin
Projects
None yet
Development

No branches or pull requests

3 participants