pluralsight-scraper icon indicating copy to clipboard operation
pluralsight-scraper copied to clipboard

Videos with resolution of 1024x768 cannot be downloaded

Open heinrich-ulbricht opened this issue 4 years ago • 4 comments

Observed behavior: Currently the script is not able to download (older) videos that have a resolution of 1024x768. The error message is not helpful: "Something went wrong. Double check the URL and try logging in again."

Expected behavior: Videos with a resolution of 1024x768 can be downloaded.

Analysis The resolution of 1280x720 is hard-baked into the code here:

https://github.com/knyzorg/pluralsight-scraper/blob/8388d58bfdb760d55f50e60ed852fbe063546ea4/index.js#L21

Changing this manually to 1024x768 works. The solution would be to try the higher resolution first, then fall back to the lower one. This is exactly what the web player does (I checked).

heinrich-ulbricht avatar Apr 12 '20 21:04 heinrich-ulbricht

I kind of expected this error to arise, but couldn't find a place where it happened. Do you have any idea where we can retrieve the available resolution information?

vezaynk avatar Apr 12 '20 21:04 vezaynk

I don't know where to get the available resolutions, but the web video player of Pluralsight does neither. It tries 1280x720 first (the 404 in the screenshot), then 1024x768:

image

So doing the same seems appropriate - with a proper delay between requests. Rate limiting seems to kick in fairly quickly.

heinrich-ulbricht avatar Apr 12 '20 21:04 heinrich-ulbricht

The resolutions seem hard-coded into the embedded player js file. Extracting it would be a pain.

The solution would be to try the higher resolution first, then fall back to the lower one.

This seems like the ideal solution. I unfortunately have a lot on my plate at the moment, but PRs are welcome!

vezaynk avatar Apr 12 '20 22:04 vezaynk

Good find. Same here, and yes :D

heinrich-ulbricht avatar Apr 12 '20 22:04 heinrich-ulbricht