Closed arabcoders closed 8 months ago
@alexta69 I've implemented the suggested fixes.
@alexta69 i started with using the same serializer however it was complaining about DownloadInfo cannot be JSON encoded
@alexta69 i started with using the same serializer however it was complaining about DownloadInfo cannot be JSON encoded
So now with the latest changes it works OK?
@alexta69 i started with using the same serializer however it was complaining about DownloadInfo cannot be JSON encoded
So now with the latest changes it works OK?
From my limited testing with sample data yeah.
Perfect. Thanks for your contribution!
Perfect. Thanks for your contribution!
Thank you. Now i wish to tackle multi download. As i download mostly live streams from youtube and usually the streamers start at or close to each other. Do you have an interest in that?
I would merge such a PR, sure. A configuration variable would need to be added that limits the number of concurrent downloads, and I think the safest would be to default to 1, and let users set it if needed, because I'm afraid that concurrent downloads opened by default might get people throttled (maybe not on youtube but on other platforms, who knows).
Hello,
I have exposed the done/queue list as JSON endpoint which helps in aiding of writing scripts to capture live streams as sometimes the added item or cronjob will miss events. Some channels i follow after the live stream ends they make the video private thus the next time the cronjob runs, it will be presented with a private video message.
I have also fixed unrelated bug that mostly happens probably in docker if you sometimes suddenly restart docker container when the container was downloading video, the queue get corrupted. This fix simply checks if the entry is actually set instead of assume it is and crashing. Example of the error this bug fixes
I have changed the url and folder to prevent copystrikes