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

unrecognized proxy format #73

Closed
ouonet opened this issue Dec 14, 2020 · 4 comments
Closed

unrecognized proxy format #73

ouonet opened this issue Dec 14, 2020 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@ouonet
Copy link

ouonet commented Dec 14, 2020

all_proxy=socks5h://127.0.0.1:1080 mainline --check

mainline 1.0.12
Distribution: Ubuntu 20.04.1 LTS
Architecture: amd64
Running kernel: 5.9.0-050900-generic
E: Caught Error while parsing environment variable 'all_proxy'
Exception: [AbstractOptionHandler.cc:69] errorCode=28 We encountered a problem while processing the option '--all-proxy'.
  -> [OptionHandlerImpl.cc:520] errorCode=1 unrecognized proxy format


E: Internet connection is not active
E: Caught Error while parsing environment variable 'all_proxy'
Exception: [AbstractOptionHandler.cc:69] errorCode=28 We encountered a problem while processing the option '--all-proxy'.
  -> [OptionHandlerImpl.cc:520] errorCode=1 unrecognized proxy format


E: Internet connection is not active
Fetching index from kernel.ubuntu.com...
OK

@bkw777
Copy link
Owner

bkw777 commented Dec 18, 2020

The issue appears to be that aria2c does not support socksv5

aria2/aria2#153

I have no proposed fix. Complain to aria2c or don't use socks5 or try to figure out how to "other ways to socksify aria2c" as mentioned in that issue, though I don't see any example directions how to actually do it.

I will add a "me too" to that thread.

@bkw777
Copy link
Owner

bkw777 commented Dec 21, 2020

Given that the aria2c ticket is actually ancient, and both wget and curl support the use case, and aria2c still doesn't 7 years after a ticket was opened for it... we have to assume this will never work with aria2c, and perhaps we should switch aout aria2c for curl or wget.

I don't think there is anything all that special we are getting from aria2c vs any other http download method. Our http download needs are simple. Even if we used a tool that had no fancy retry strategies built in, we could just do that in a shell script or directly in the vala main app code perfectly easily.

I will at least take a look at swapping out aria2c for curl. Or maybe an external but bundled shell script, so that the app can use any downloader as long as you write a wrapper script around it to implement a consistent interface with the app. So we could bundle 3 scripts to begin with, one each for wget, curl, and aria2c, and the user can select which one they want in the config screen. And for really special needs, the user can copy and modify one of the scripts to make a custom one to do whatever they need to deal with extra unusual network needs, without needing the app to support unpredictable new situations.

We could probably use libsoup or libcurl and have the http downloading built in, but an external wrapper/driver script lets the user handle upredictable new situations if they need to.

@bkw777 bkw777 added the enhancement New feature or request label Dec 21, 2020
@bkw777 bkw777 self-assigned this Dec 21, 2020
@bkw777
Copy link
Owner

bkw777 commented Aug 7, 2021

I just noticed that there are http-proxy options that may be specified in download list files
https://aria2.github.io/manual/en/html/aria2c.html#id2

And we do in fact use a list file for downloads in DownloadManager.vala
So maybe we add a few config options that will get added here?

string list_file = working_dir+"/download.list";

@bkw777
Copy link
Owner

bkw777 commented Mar 14, 2023

Proxy support has been added.

@bkw777 bkw777 closed this as completed Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants