-
Notifications
You must be signed in to change notification settings - Fork 367
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
[zoom] new extractor #108
[zoom] new extractor #108
Conversation
…t throw warning if password field not found
seems to be working fine with all the Zoom links I've tried. Public and a couple password protected. Good job! |
shouldn't this extractor be called |
yup |
While this awesome extractor seems to be working fine, I've come across this password protected link today and it seems to have some custom theme on the page. Extractor does not see or identify it as one it can handle. Don't have a password to it yet to test if it works with --password parameter forced. Please see the log below: |
Before submitting a pull request make sure you have:
In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:
What is the purpose of your pull request?
Fixes #107
Original pull request: https://github.com/unqueued/youtube-dl-prs/tree/pr/24751
I think the password field detection needs updating, however it downloads public videos.