Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support # in the URL #10

Closed
ksy5662 opened this issue Dec 21, 2013 · 2 comments
Closed

Support # in the URL #10

ksy5662 opened this issue Dec 21, 2013 · 2 comments

Comments

@ksy5662
Copy link

ksy5662 commented Dec 21, 2013

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/stubedown63772/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/stubedown63772/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
Copy link
Owner

jaimeMF commented Dec 21, 2013

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.

@jaimeMF jaimeMF closed this as completed Dec 21, 2013
@ksy5662
Copy link
Author

ksy5662 commented Dec 21, 2013

Thanks a lot!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants