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

generic extractor improvement: add a JavaScript beautifier #5913

Closed
eadmaster opened this issue Jun 7, 2015 · 5 comments
Closed

generic extractor improvement: add a JavaScript beautifier #5913

eadmaster opened this issue Jun 7, 2015 · 5 comments

Comments

@eadmaster
Copy link

@eadmaster eadmaster commented Jun 7, 2015

A JavaScript beautifier could help with websites obfuscating direct video URLs.

@dstftw dstftw closed this Jun 7, 2015
@yan12125
Copy link
Collaborator

@yan12125 yan12125 commented Jun 7, 2015

I'm not sure why @dstftw closed this issue. AFAIK, tools like beautifiers help humans but not machines. In other words, they help programmars but not programs (youtube-dl), so there's no need to integrate the tools into youtube-dl.

@eadmaster
Copy link
Author

@eadmaster eadmaster commented Jun 7, 2015

the point is that some video hosting websites are obfuscating the direct video URL using javascript. So a JavaScript beautifier could expose/un-obfuscate the url for downloading with the generic extractor.

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Jun 7, 2015

You'll have to post an example url.
Implementing a general beautifier in python is probably a big task, it would be better if we had an example we can start with.

@eadmaster
Copy link
Author

@eadmaster eadmaster commented Jun 7, 2015

You could take this as an example:
http://www.rapidvideo.org/m612rwnlbv5k
(even if the website won't be supported officially due to licensed contents, it could be still supported using the generic plugin)

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Jun 7, 2015

We'd need to add a test for it, so please post a url from a site that doesn't infringe copyright.

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

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.