Bug#655667: get-flash-videos: fails on URLs containing UTF-8 characters

[Marc Dequènes (Duck)]

Package: get-flash-videos
Version: 1.25~git2011.09.26-2
Tags: patch


Coin,

For historical purpose i was trying to archive the following video but it failed: $ get_flash_videos "http://www.youtube.com/watch?v=kL5DDSglM_s&feature=related";

Downloading http://www.youtube.com/watch?v=kL5DDSglM_s&feature=related

Using method 'youtube' for http://www.youtube.com/watch?v=kL5DDSglM_s&feature=related Error: Can't escape \x{304D}, try uri_escape_utf8() instead at /usr/share/perl5/FlashVideo/Site/Youtube.pm line 393

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



Attached is a proper patch for this issue.

Regards.

--
Marc Dequènes (Duck)

--- /usr/share/perl5/FlashVideo/Site/Youtube.pm.orig	2012-01-13 02:26:31.395727309 +0100
+++ /usr/share/perl5/FlashVideo/Site/Youtube.pm	2012-01-13 02:26:48.099939882 +0100
@@ -391,7 +391,7 @@
       "http://www.youtube.com/get_video_info?&video_id=%s&el=$el&ps=default&eurl=%s&hl=en_US&t=%s";;
 
     my $video_info_url = sprintf $video_info_url_template,
-      uri_escape($video_id), uri_escape($url), uri_escape($t);
+      uri_escape_utf8($video_id), uri_escape_utf8($url), uri_escape_utf8($t);
 
     debug "get_youtube_video_info: $video_info_url";
 

I tested replacing the three uri_escape() calls on this page by uri_escape_utf8(), as suggested in the error message, and it worked like a charm. Maybe we could had such a nice resolution suggestion feature to all packages and for once get RC bugs down to 0 ;-).

pgpFo649g9fql.pgp
Description: PGP Digital Signature