danvics/ytdl-sub
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(docker): Unintentional unattended dry runs

Browse source 
[PR feedback](https://github.com/jmbannon/ytdl-sub/pull/1321#discussion_r2315215600)
prompted me to reconsider having a default command at all. We should assume,
unfortunately, that many new users will just skim the docs enough to enable the image's
cron integration but not actually incrementally test their configuration. In those
cases, they'd end up sending dry-run non-download requests for all their subscriptions
every 6 hours for no good reason. There's just no way to provide a default command that
isn't also providing a footgun.
This commit is contained in:
Ross Patterson 2025-09-02 02:48:43 -07:00
parent c1541ab904
commit 9e7ac3777d
No known key found for this signature in database
GPG key ID: 2EFF7CCE6828E359
1 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
docker/root/defaults/cron
View file

@ -4,9 +4,12 @@
# default `./config.yaml` and `./subscriptions.yaml`, so you don't need to use the
# `--config` CLI option or pass a `SUBPATH` to the `$ ytdl-sub sub` sub-command.
#
# To prevent users accidentally triggering throttles or bans or downloading before
# testing their configuration, these default options only simulate a few
# downloads. Remove the `--dry-run` and `-o ...` CLI options when you've tested your
# configuration and you're ready to download entries unattended:
ytdl-sub --dry-run sub -o '--ytdl_options.max_downloads 3'
# Test your configuration and subscriptions carefully before automating downloads to
# prevent triggering throttles or bans:
#
# https://ytdl-sub.readthedocs.io/en/latest/guides/getting_started/downloading.html
#
# Once you've tested your configuration and you're ready to download entries unattended,
# remove the next line and un-comment the following line:
echo "WARNING: Read /config/ytdl-sub-configs/cron and modify to automate downloads."
# ytdl-sub sub