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

Added options to include cookies for ytld #33

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

samusaran
Copy link

This will enable request to age-restricted videos

This will enable request to age-restricted videos
Copy link
Owner

@Androz2091 Androz2091 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it still working when specifying an empty cookie or an invalid cookie?

@samusaran
Copy link
Author

I added a coalesce just to avoid the system crashing if the COOKIES parameter is missing completely.
I can't check if the cookies are actually not well formed.

It works fine if the parameter is present but empty.

@Androz2091 Androz2091 self-requested a review September 18, 2021 07:16
@clifford64
Copy link

Can you provide an example on how to call a cookie.txt file within .env?

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.

3 participants