jaimeMF / youtube-dl-api-server

A youtube-dl REST API server
https://youtube-dl-api-server.readthedocs.org/
The Unlicense
364 stars 212 forks source link

Support # in the URL #10

Closed ksy5662 closed 10 years ago

ksy5662 commented 10 years ago

Seems not taking # in the URL. commandline youtube-dl is O.K with some changed for blinkx

_VALID_URL = r'^(?:https?://(?:www.)blinkx.com/#*ce/|blinkx:)(?P[^?]+)'

Here is some error log for GAE

http://xxxxxx.appspot.com/api/?url=http://www.blinkx.com/#ce/8aQUy7GVFYgFzpKhT0oqsilwOGFRVXk3R1ZGWWdGenBLaFQwb3FzaWx3OGFRVXk3R1ZGWWdGenB

Log from GAE

2013-12-21 05:13:39.180 /api/?url=http://www.blinkx.com/ 200 464ms 0kb Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/31.0.1650.63 Safari/537.36 module=default version=alpha-3-4 221.158.56.138 - - [21/Dec/2013:05:13:39 -0800] "GET /api/?url=http://www.blinkx.com/ HTTP/1.1" 200 321 - "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/31.0.1650.63 Safari/537.36" "tubedown63772.appspot.com" ms=465 cpu_ms=47 cpm_usd=0.000099 app_engine_release=1.8.8 instance=00c61b117c281cde63f98bd4668e70a5a95bfb60 E 2013-12-21 05:13:38.936 WARNING: Falling back on generic information extractor. D 2013-12-21 05:13:38.937 [generic] : Downloading webpage D 2013-12-21 05:13:39.173 [generic] : Extracting information E 2013-12-21 05:13:39.177 ERROR: Unsupported URL: http://www.blinkx.com/; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type youtube-dl -U to update. E 2013-12-21 05:13:39.179 Traceback (most recent call last): File "/base/data/home/apps/s~tubedown63772/alpha-3-4.372489474374418417/youtube_dl_server/API/APIresponder.py", line 73, in get_response vids = videos(url) File "/base/data/home/apps/s~tubedown63772/alpha-3-4.372489474374418417/youtube_dl_server/API/APIresponder.py", line 49, in videos res = ydl.extract_info(url, download=False) File "/base/data/home/apps/s~tubedown63772/alpha-3-4.372489474374418417/lib/youtube_dl/YoutubeDL.py", line 496, in extract_info self.report_error(compat_str(de), de.format_traceback()) File "/base/data/home/apps/s~tubedown63772/alpha-3-4.372489474374418417/lib/youtube_dl/YoutubeDL.py", line 368, in report_error self.trouble(error_message, tb) File "/base/data/home/apps/s~tubedown63772/alpha-3-4.372489474374418417/lib/youtube_dl/YoutubeDL.py", line 343, in trouble raise DownloadError(message, exc_info) DownloadError: ERROR: Unsupported URL: http://www.blinkx.com/; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type youtube-dl -U to update.

jaimeMF commented 10 years ago

You have to encode the url, it should be url=http%3A%2F%2Fwww.blinkx.com%2F%23ce%2F8aQUy7GVFYgFzpKhT0oqsilwOGFRVXk3R1ZGWWdGenBLaFQwb3FzaWx3OGFRVXk3R1ZGWWdGenB. It will depend on the language you're using, in python you can use urllib.urlencode. Most url wont have any problems, but others will, so you should always escape them.

ksy5662 commented 10 years ago

Thanks a lot!!