-
Notifications
You must be signed in to change notification settings - Fork 289
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
Include other options in string? #29
Comments
In order to include that options,
after that, re-start the docker-container. |
Has anyone tried running with the ignore error flag enabled? Wondering if there is any harm in just enabling it by default. We don't really report any errors back to the user anyway. |
I learned this flag does not work in python based on this docker. I think we need some modification that the python script restarts automatically in case of happening any exception or error. |
Probably just need to catch and handle errors on that thread. Do you have a playlist you were testing with? |
https://www.youtube.com/watch?v=wNtbfEaAw18&list=PLofOAS0adMaxoTfZf7HLJ5Vk3As5JuDtE&index=2 The link above is what I was testing. The fourth clip was deleted. So, when I tried to download it, the error happened and the process was suspended at that point. Then I tried downloading another clip(even very normal) but I couldn't do until I restarted the docker container. |
I've just found the solution. After that, --ignore-errors works so well and youtube-dl-server.py weren't suspended anymore. |
Do you think I should make this the default setting? |
Yes, when we try to download playlists, oftentimes we experience that kind of error. Otherwise, we will always have to restart the docker-container whenever we meet any exception or error of python. |
I noticed that YouTube-dl-server will stop if there is a private video listed YT playlist. Is there a way I can include —ignore-errors
The text was updated successfully, but these errors were encountered: