[BACKEND]: Less finger-printable request delay in throttle protection (#1273)
Thanks @rpatterson for the fix! --------- Following [discussion about the previous preset delay](https://discord.com/channels/994270357957648404/994270357957648408/1405705039649046589), we weren't able to find a reason for it and there was agreement that the example value from yt-dlp is more sensible. While we're at it, also set a default `min:` value that would probably be sensible if it were to be used in the future. Finally, capture the reasoning behind the new preset values in comments to make the reference documentation more helpful to new users in the future.
This commit is contained in:
parent
2f55326981
commit
47dc861727
1 changed files with 10 additions and 3 deletions
|
|
@ -14,8 +14,15 @@ presets:
|
|||
)
|
||||
}
|
||||
sleep_per_request_s:
|
||||
min: 3.5
|
||||
max: 3.5
|
||||
# yt-dlp processes eath request synchronously, so it's intrinsic delay between
|
||||
# requests already represent a maximum close to real app/client behavior in the
|
||||
# field. Add a small additional margin matching the example values from yt-dlp
|
||||
# to be conservative:
|
||||
max: 0.75
|
||||
# Not used at time of writing, but choose a value that would mimic real
|
||||
# app/client behavior in the field. The next request is usually sent right away
|
||||
# if processing the previous request is done asynchronously:
|
||||
min: 0.0
|
||||
sleep_per_download_s:
|
||||
min: 13.8
|
||||
max: 28.4
|
||||
|
|
@ -23,4 +30,4 @@ presets:
|
|||
min: 16.3
|
||||
max: 26.1
|
||||
overrides:
|
||||
enable_throttle_protection: True
|
||||
enable_throttle_protection: True
|
||||
|
|
|
|||
Loading…
Reference in a new issue