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

[Minds] Add new extractor #17934

Merged
merged 8 commits into from
Jan 17, 2021
Merged

[Minds] Add new extractor #17934

merged 8 commits into from
Jan 17, 2021

Conversation

Tatsh
Copy link
Contributor

@Tatsh Tatsh commented Oct 21, 2018

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

This adds support for Minds' channel pages, single newsfeed pages, and media pages which play a video. Fixes issue #13220.

This requires cookies to work even if not logged as the XSRF-TOKEN string must be used with all API calls. The x-xsrf-token header value used in API calls is taken from cookies. If you have your XSRF taken from the site, you can save it to a file like so:

printf '.minds.com\tTRUE\t/\tTRUE\t15624050615\tXSRF-TOKEN\t< TOKEN VALUE HERE >\n' >> ~/.ytdl-minds-cookies

Then use cookies:

youtube-dl --cookies ~/.ytdl-minds-cookies 'https://www.minds.com/ottman'

This closes #13220

@Tatsh
Copy link
Contributor Author

Tatsh commented Jan 26, 2019

@dstftw Please review again since the last commit.

@rudolphos
Copy link

Are there any updates to this?

using latest version

ERROR: Unsupported URL: https://www.minds.com/newsfeed/..

@Tatsh
Copy link
Contributor Author

Tatsh commented Mar 1, 2019

The general newsfeed is not supported in this PR but that would not be too hard to support. Only /newsfeed/<some ID here> newsfeed URLs are supported.

@Tatsh
Copy link
Contributor Author

Tatsh commented Mar 20, 2019

Bump

@dstftw dstftw force-pushed the master branch 2 times, most recently from 5e26784 to da2069f Compare September 13, 2020 13:51
@Tatsh
Copy link
Contributor Author

Tatsh commented Nov 17, 2020

@dstftw If you want me to make changes, I'll need to re-make this PR. I asked GitHub and they cannot fix branch references.

@remitamine remitamine merged commit de026a6 into ytdl-org:master Jan 17, 2021
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for Minds.com
4 participants