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

[BACKEND] Make resolution print debug-level (#1326)

Browse source 
```
Resolution assert is enabled, will fail on low-quality video downloads and presume throttle. Disable using the override variable `enable_resolution_assert: False
```
Spams during the initialization phase. Ideally, we should only print this once the subscription has begun downloading. Perhaps via a `print` plugin? Either way, set this to debug log level.
This commit is contained in:
Jesse Bannon 2025-09-01 14:33:46 -07:00 committed by GitHub
parent 3eb838eb40
commit 2e67a57bc9
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/ytdl_sub/prebuilt_presets/helpers/throttle_protection.yaml
View file

@ -43,7 +43,8 @@ presets:
"Resolution assert is enabled, will fail on low-quality video downloads and presume throttle. Disable using the override variable `enable_resolution_assert: False`", "Resolution assert is enabled, will fail on low-quality video downloads and presume throttle. Disable using the override variable `enable_resolution_assert: False`",
"Resolution assert is disabled. Use at your own risk!" "Resolution assert is disabled. Use at your own risk!"
), ),
enable_resolution_assert enable_resolution_assert,
-1
) )
} }
resolution_readable: "{width}x{height}" resolution_readable: "{width}x{height}"

4
tests/integration/plugins/test_throttle_protection.py
View file

@ -205,7 +205,7 @@ class TestResolutionAssert:
with assert_logs( with assert_logs(
logger=script_print_logger, logger=script_print_logger,
expected_message="Resolution assert is disabled. Use at your own risk!", expected_message="Resolution assert is disabled. Use at your own risk!",
log_level="info", log_level="debug",
expected_occurrences=1, expected_occurrences=1,
): ):
_ = Subscription.from_dict( _ = Subscription.from_dict(
@ -237,7 +237,7 @@ class TestResolutionAssert:
"Resolution assert is enabled, will fail on low-quality video downloads and presume throttle. " "Resolution assert is enabled, will fail on low-quality video downloads and presume throttle. "
"Disable using the override variable `enable_resolution_assert: False`" "Disable using the override variable `enable_resolution_assert: False`"
), ),
log_level="info", log_level="debug",
expected_occurrences=1, expected_occurrences=1,
): ):
subscription = Subscription.from_dict( subscription = Subscription.from_dict(