search

giavoni / get-flash-videos

Automatically exported from code.google.com/p/get-flash-videos
0 stars 0 forks source link

Hulu Plug-In Not Working #333

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?

get_flash_videos-1.24 http://www.hulu.com/watch/3
00844/law-and-order-special-victims-unit-educated-guess#s-p1-so-i0 --debug

What is the expected output? What do you see instead?

Expected: Video download begins.

What I see:

~1~ ~p~l~u~g~i~n~ ~i~n~s~t~a~l~l~e~d~:~
~~-~ ~H~u~l~u~.~p~m~
~Downloading 
http://www.hulu.com/watch/300844/law-and-order-special-victims-unit-educated-gue
ss#s-p1-so-i0
-> GET 
http://www.hulu.com/watch/300844/law-and-order-special-victims-unit-educated-gue
ss#s-p1-so-i0
<- 200 text/html; charset=utf-8 (229276): UTF8 on, non-ASCII, 229276 characters
229328 bytes
Trying to open plugin ~/get_flash_videos/plugins/Www.pm
Trying to open plugin ~/get_flash_videos/plugins/Hulu.pm
Using method 'hulu' for 
http://www.hulu.com/watch/300844/law-and-order-special-victims-unit-educated-gue
ss#s-p1-so-i0
Found Hulu CID: 60004804
Returning cached Hulu data
-> GET http://r.hulu.com/videos?content_id=60004804
<- 404 text/plain (13): UTF8 off, ASCII, 13 characters 13 bytes
Error: Couldn't parse Hulu XML: File does not exist: 404 Not Found at 
/loader/0x2383b54/FlashVideo/Site/Hulu.pm line 72

Couldn't extract Flash movie URL. This site may need specific support adding,or 
fixing.

Please confirm the site is using Flash video and if you have Flash available 
check that the URL really works(!).

Check for updates by running: get_flash_videos-1.24 --update

If the latest version does not support this please open a bug (or
contribute a patch!) at http://code.google.com/p/get-flash-videos/
make sure you include the output with --debug enabled.
Couldn't download any videos.

What version of the product are you using? On what operating system?

get_flash_videos version 1.24 
Windows XP

Please provide any additional information below.

Original issue reported on code.google.com by annoyw...@gmail.com on 11 Dec 2011 at 4:30

GoogleCodeExporter commented 9 years ago 
Finally somebody reported this bug. I thought I was the only one that 
encountered this bug.

It looks like that Hulu has changed the "videos" program on their 
http://r.hulu.com/ that it no longer takes "content_id" as a acceptable 
variable. It apparently replies 404 error when illegal arguments were passed to 
it.

$ wget -U Mozilla -q -O - 
http://www.hulu.com/watch/300844/law-and-order-special-victims-unit-educated-gue
ss |grep content_id
    so.addVariable("content_id", 60004804);
$ wget -U Mozilla -q -S -O - http://r.hulu.com/videos?content_id=60004804
  HTTP/1.0 404 Not Found
  Retry-After: 0
  Content-Type: text/plain
  ntCoent-Length: 13
  X-Varnish: 1017115836
  Server: Varnish
  Cache-Control: private
  Date: Sun, 11 Dec 2011 06:27:35 GMT
  Content-Length: 13
  Connection: keep-alive

By doing some digging tonight, fortunately, I found that it still accepts the 
"eid" argument:
$ wget -U Mozilla -q -O - 
http://www.hulu.com/watch/300844/law-and-order-special-victims-unit-educated-gue
ss |grep 'embed src=' |sed -e 's/.*embed src="\([^"]\+\)".*/\1/g'
http://www.hulu.com/embed/BOb38xCgmJxccy0nScbznA
$ wget -U Mozilla -q -O - http://r.hulu.com/videos?eid=BOb38xCgmJxccy0nScbznA 
|sed -e 's|.*\(<pid>.*</pid>\).*|\1\n|g'
<pid>NO_MORE_RELEASES_PLEASE_60004804</pid>

Based on my finding, I created a patch for a quick fix. Again, this patch is 
kind of a dirty hack, but it works for me,

Original comment by storm.sf...@gmail.com on 11 Dec 2011 at 6:55

Attachments:

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago 
Patch works great!  Many thanks.

Original comment by annoyw...@gmail.com on 12 Dec 2011 at 1:19

GoogleCodeExporter commented 9 years ago 
Yeah, patch worked for me as well. Thanks!

Original comment by gesquive on 12 Dec 2011 at 1:22

GoogleCodeExporter commented 9 years ago 
Fixed in gitorious by merging a pull request on there. Run get_flash_videos 
--update to get the latest version.

Storm: thanks for your fix, in future please clone on gitorious 
(https://gitorious.org/get-flash-videos-plugins) and submit a merge request 
there as it's easier to merge and you can get credited for the fix (if you 
care). Please also comment on here though so people can find it. Also let me 
know if want to be added to gfv-hackers on gitorious so you push directly.

Original comment by zakflash...@gmail.com on 13 Dec 2011 at 9:49

GoogleCodeExporter commented 9 years ago 
Roger that and will do. I've just created my account on Gitorious. It probably 
will take me some time to get used to the tools provided by Gitorious.

Original comment by storm.sf...@gmail.com on 14 Dec 2011 at 11:50