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

[tiktok] rewrite for new API and website #26281

Closed
wants to merge 1 commit into from
Closed

[tiktok] rewrite for new API and website #26281

wants to merge 1 commit into from

Conversation

StephenFluin
Copy link

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:

  • I am the original author of this code and I am willing to release it under Unlicense
  • I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

  • Bug fix
  • Improvement
  • New extractor
  • New feature

Description of your pull request and other information

Fix the TikTok extractor by using the raw HTML from the page.

You should consider merging #25895 instead, but it is possibly more fragile as it relies on unwatermarked older APIs.

@megapro17
Copy link

It will extract watermarked h264 stream? Tiktok can give h265 tho

@MylesBorins MylesBorins mentioned this pull request Sep 7, 2020
5 tasks
@MylesBorins
Copy link

I floated this PR locally and manually tested to download a video and it worked!

Also hi @StephenFluin o/

@cardoso-neto
Copy link

What's holding this pull request back?

@dstftw dstftw force-pushed the master branch 2 times, most recently from 5e26784 to da2069f Compare September 13, 2020 13:50
@dstftw dstftw closed this in fb626c0 Nov 29, 2020
ThirumalaiK pushed a commit to ThirumalaiK/youtube-dl that referenced this pull request Jan 28, 2021
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

Successfully merging this pull request may close these issues.

4 participants