Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Support allow_paid_broadcast in AIORateLimiter #4578

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.

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

Closed
Bibo-Joshi opened this issue Nov 25, 2024 · 0 comments · Fixed by #4627
Closed

Support allow_paid_broadcast in AIORateLimiter #4578

Bibo-Joshi opened this issue Nov 25, 2024 · 0 comments · Fixed by #4627
Labels
ℹ️ needs-wiki-update information: needs-wiki-update

Comments

@Bibo-Joshi
Copy link
Member

What kind of feature are you missing? Where do you notice a shortcoming of PTB?

AIORateLimiter is currently not considereding allow_paid_broadcast which means that this parameter has no effect when using the limiter.

Describe the solution you'd like

Extract the value for allow_paid_broadcast in process_update. In case it's True, the update should be sent to a standalone aiolimiter instance that allows up to 1000 messages per second.

Describe alternatives you've considered

No response

Additional context

#4573 adds a warning about missing support for now.
https://github.com/python-telegram-bot/python-telegram-bot/wiki/Avoiding-flood-limits also contains a warning, that should be removed if this is implemented

@Bibo-Joshi Bibo-Joshi added 📋 triage work status: triage ℹ️ needs-wiki-update information: needs-wiki-update and removed 📋 triage work status: triage labels Nov 25, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Jan 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ℹ️ needs-wiki-update information: needs-wiki-update
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant