Skip to content
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

feat: use lib side check of valid lengths and prevent invalid requests being sent. #124

Open
wants to merge 11 commits into
base: dev
Choose a base branch
from

Conversation

anonymousx97
Copy link
Contributor

@anonymousx97 anonymousx97 commented Dec 15, 2024

New Checks in:

  • send_message
  • set_administrator_title
  • update_profile
  • answer_inline_query

…s being sent.

New Checks in:
send_message
edit_message text/caption
set_administrator_title
update_profile (bio)
@anonymousx97
Copy link
Contributor Author

The Current approach checks length of text with length of entities included.

I do not know how pyrogram handles those so adjust the check_valid_length function to allow checking for length after entities are parsed.

@SpEcHiDe SpEcHiDe marked this pull request as draft December 24, 2024 03:48
@SpEcHiDe SpEcHiDe marked this pull request as ready for review December 24, 2024 13:48
@SpEcHiDe SpEcHiDe changed the base branch from L195 to dev December 27, 2024 17:47
@SpEcHiDe SpEcHiDe mentioned this pull request Feb 15, 2025
29 tasks
SychO3 pushed a commit to SychO3/neogram that referenced this pull request Mar 6, 2025
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