-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
[ie/Boosty] include auth token in API call if available #9522
Conversation
I've just checked and can confirm that this works 👍 |
72a8e24
to
2fae8c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
I've had no luck with this, tested on Windows and MacOS. |
@DocFreeman you pulled the branch and ran from source? can you show a verbose log please? |
Yes, for MacOS I did that. For Windows I just pulled a nightly build to see if it could partially parse it at least but it could not.
|
This PR specifically only fixes paywalled |
It's working! Thanks a lot! 🤝 |
Fix extracting video URLs from paywalled posts on Boosty
For paywalled posts (posts that require a subscription level), Boosty's API only returns the video URLs if the access token is set. This change attempts to extract the token from the provided cookies and sets the authorization header for the API call accordingly.
Fixes #9401
Template
Before submitting a pull request make sure you have:
In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:
What is the purpose of your pull request?