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

Bump tusd to v2.4.0 #4784

Merged
merged 1 commit into from
Aug 2, 2024
Merged

Bump tusd to v2.4.0 #4784

merged 1 commit into from
Aug 2, 2024

Conversation

kobergj
Copy link
Contributor

@kobergj kobergj commented Jul 30, 2024

Bumps tusd to v2.4.0 to enable tackling owncloud/ocis#1755

@kobergj kobergj requested review from a team and labkode as code owners July 30, 2024 11:57
Copy link

update-docs bot commented Jul 30, 2024

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@kobergj
Copy link
Contributor Author

kobergj commented Jul 30, 2024

Full-ci ocis pipeline is green: owncloud/ocis#9714

@2403905
Copy link

2403905 commented Jul 30, 2024

If I understand correctly we also have to implement the Unwrap interface otherwise we getting warnings regarding NetworkControlError/NetworkTimeoutError and "feature not supported"

https://tus.github.io/tusd/advanced-topics/usage-package/#i-am-getting-warnings-regarding-networkcontrolerrornetworktimeouterror-and-feature-not-supported-why

pkg/rhttp/datatx/manager/tus/tus.go Outdated Show resolved Hide resolved
Signed-off-by: jkoberg <jkoberg@owncloud.com>
@kobergj
Copy link
Contributor Author

kobergj commented Aug 1, 2024

If I understand correctly we also have to implement the Unwrap interface otherwise we getting warnings regarding NetworkControlError/NetworkTimeoutError and "feature not supported"

Correct, but we are passing a noop logger (with did that before too). Hence we will not see those error messages.

@kobergj kobergj merged commit 0944995 into cs3org:edge Aug 2, 2024
9 of 10 checks passed
@kobergj kobergj deleted the BumpTusdPkg branch August 2, 2024 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants