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

Added option to download through a proxy #308

Merged
merged 7 commits into from
Oct 30, 2022
Merged

Conversation

Diiak
Copy link
Contributor

@Diiak Diiak commented Oct 18, 2022

In connection with the issue I created. I added the option to download through a proxy.

It works with HTTP and SOCKS5 proxies.

For HTTP : spotify_dl -p "http://username:password@hostname:port" or "http://hostname:port"
For SOCKS5 : spotify_dl -p "socks5://username:password@hostname:port" or "socks5://hostname:port"

I hope this will help some people 🚀

@Diiak
Copy link
Contributor Author

Diiak commented Oct 18, 2022

@SathyaBhat Let me know if it's good for you. I saw that there was a problem with the deep analysis but I checked my indentation is correct 😃

@SathyaBhat
Copy link
Owner

Hi @Diiak - the deepsource complaint is that you use the tab character instead of spaces. Can you correct that? https://deepsource.io/gh/SathyaBhat/spotify-dl/run/3b926209-6a70-43a4-a1a7-7c369c234fb4/python/

Also, could you please update the readme & getting started about the proxy flag?

Thanks!

spotify_dl/youtube.py Outdated Show resolved Hide resolved
Co-authored-by: Sathyajith Bhat <sathya@sathyasays.com>
@Diiak
Copy link
Contributor Author

Diiak commented Oct 24, 2022

@SathyaBhat I don't understand why the download doesn't work. What is the problem? It doesn't seem to be related to the proxy since the tests don't use one 🤔

@SathyaBhat
Copy link
Owner

it looks like sponsorblock API is causing troubles. Need to check deeper

@Diiak
Copy link
Contributor Author

Diiak commented Oct 24, 2022

it looks like sponsorblock API is causing troubles. Need to check deeper

I was also having problems when testing locally with sponsorblock but the sound was still downloading. Let me know when everything is ok then!

@SathyaBhat
Copy link
Owner

the download will work, sponsorblock API is used to fetch skippable segments and then the file is converted to mp3. If this is skipped, the file will be downloaded but not converted to mp3. The tests specifically check for this as well, not just the download and hence it fails

@Diiak
Copy link
Contributor Author

Diiak commented Oct 24, 2022

the download will work, sponsorblock API is used to fetch skippable segments and then the file is converted to mp3. If this is skipped, the file will be downloaded but not converted to mp3. The tests specifically check for this as well, not just the download and hence it fails

I understand the concern then. I see on the status page that there are indeed some issues. We'll do the tests again when it's fixed 👍

@Diiak
Copy link
Contributor Author

Diiak commented Oct 29, 2022

@SathyaBhat Hello, It looks like the sponsorblock server is available today. Can we try the tests again?

@SathyaBhat
Copy link
Owner

SathyaBhat commented Oct 29, 2022

@Diiak seems like its still having issues

WARNING: HTTP Error 502: Bad Gateway. Retrying...
WARNING: HTTP Error 502: Bad Gateway. Retrying...
WARNING: HTTP Error 502: Bad Gateway. Retrying...
ERROR: Postprocessing: Unable to communicate with SponsorBlock API: HTTP Error 502: Bad Gateway

would be a good time to fix skip sponsorblock #309

@SathyaBhat SathyaBhat merged commit 5431943 into SathyaBhat:master Oct 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants