-
Notifications
You must be signed in to change notification settings - Fork 88
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
Major release: v2 #78
Comments
Hey, @Acconut , could you share an ETA for this release? |
Sorry, we do not have any schedule available to share. Explicit work on this has not yet started. |
Hi @Acconut, Would it be possible to add support for proxies earlier than v2 if I were to contribute code in a PR? I don't wish to override code in multiple places or use system settings to accomplish this. I believe the change would be rather simple if we use java.net.Proxy.NO_PROXY as a default. |
Yes, of course! Feel free to open a PR for this. |
@Acconut we could fix this pretty quickly and robust. Would you be open to review the code (provided we create a PR), merge it (after comments are resolved) and create a new release? The lack of this feature is blocking our implementation. We're behind a proxy for external connections, but don't want to use it for all request as most of them are local. As said in #52 (comment). |
Yes, sure. We are always happy to accept PRs. |
Any ETA on this? This library is currently unusable with cloudflare because of the HTTP client apparently. |
No ETA, but we will bump it up on our priority list. |
Major changes:
uploadChunk()
with start and stop methods similar to tus-js-client (remove chunk size as well)HttpURLConnection
with more modern HTTP client (maybe okhttp or HttpClient?)Additional features:
The text was updated successfully, but these errors were encountered: