Skip to content

fix(youtube): treat upcoming 'needs_auth' videos as public#10

Merged
jim60105 merged 3 commits into
masterfrom
dev
Dec 17, 2025
Merged

fix(youtube): treat upcoming 'needs_auth' videos as public#10
jim60105 merged 3 commits into
masterfrom
dev

Conversation

@jim60105
Copy link
Copy Markdown
Member

No description provided.

jim60105 and others added 3 commits December 17, 2025 21:39
- Enable verbose mode and add an extractor argument (youtubepot-bgutilscript) to the youtube-dl options.

Signed-off-by: Jim Chen <Jim@ChenJ.im>
…ockerfile

- Modified Dockerfile ENTRYPOINT to start bgutil-pot server in background and LivestreamRecorderService in foreground using dumb-init for proper signal handling
- Removed ExtractorArgs 'youtubepot-bgutilscript' from YoutubeDLHelper.cs as bgutil-pot is now managed separately in the container

Signed-off-by: Jim Chen <Jim@ChenJ.im>
- Treats videos with a "needs_auth" copyright notice as public when their LiveStatus is "is_upcoming", adding a fall-through to the public case to work around upcoming videos being reported as needs_auth.

Signed-off-by: Jim Chen <Jim@ChenJ.im>
@jim60105 jim60105 merged commit 1d7b34f into master Dec 17, 2025
2 of 5 checks passed
@jim60105 jim60105 deleted the dev branch December 17, 2025 17:36
@deepsource-io
Copy link
Copy Markdown
Contributor

deepsource-io Bot commented Dec 17, 2025

Here's the code health analysis summary for commits af28d05..19adf10. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource C# LogoC#❌ Failure
❗ 3 occurences introduced
🎯 1 occurence resolved
View Check ↗
DeepSource Docker LogoDocker✅ SuccessView Check ↗
DeepSource Secrets LogoSecrets✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@jim60105 jim60105 restored the dev branch December 17, 2025 17:36
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.

2 participants