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.

Sign up for GitHub

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

Jump to bottom

[Enhancement] Add option for a source to only use cookies when needed #640

Merged
merged 5 commits into from
Mar 5, 2025
Merged

[Enhancement] Add option for a source to only use cookies when needed #640

merged 5 commits into from
Mar 5, 2025

Conversation

kieraneglin
Copy link
Owner

@kieraneglin kieraneglin commented Mar 5, 2025 •
edited
Loading

What's new?

  • Adds an option to sources to only use cookies when needed
    • This is useful for when you need to use cookies to access a playlist in the first place (like your Watch Later) playlist but the download of the videos itself doesn't need cookies
    • For now "when needed" only uses cookies for slow indexing. That said, I'm thinking of other ways this can be used
    • Resolves [FR] Fetching WL Playlist w/o using cookies for download #595

What's changed?

N/A

What's fixed?

N/A

Any other comments?

N/A

@kieraneglin kieraneglin changed the title [Enhancement [Enhancement] Add option for a source to only use cookies when indexing Mar 5, 2025
@kieraneglin kieraneglin self-assigned this Mar 5, 2025
@kieraneglin kieraneglin added the enhancement New feature or request label Mar 5, 2025
@kieraneglin kieraneglin marked this pull request as ready for review March 5, 2025 23:32
@kieraneglin kieraneglin changed the title [Enhancement] Add option for a source to only use cookies when indexing [Enhancement] Add option for a source to only use cookies when needed Mar 5, 2025
@kieraneglin kieraneglin merged commit ac89594 into master Mar 5, 2025
1 check passed
@kieraneglin kieraneglin deleted the ke/different-cookie-behaviours branch March 5, 2025 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@kieraneglin kieraneglin

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FR] Fetching WL Playlist w/o using cookies for download
1 participant
@kieraneglin