Skip to content
This repository was archived by the owner on Dec 23, 2024. It is now read-only.

Correct username parsing #1226

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Correct username parsing #1226

wants to merge 1 commit into from

Conversation

KyMaP13
Copy link
Contributor

@KyMaP13 KyMaP13 commented Feb 28, 2023

Its very important when you have more then one username
Usernames

@KyMaP13
Copy link
Contributor Author

KyMaP13 commented Feb 28, 2023

@delivrance hi, it helps to define the username correctly, check this please.

@KyMaP13
Copy link
Contributor Author

KyMaP13 commented Mar 2, 2023

I've tested this behavior on pyrogram version 2.0.27 and it works correctly for persons with alternative logins, username is not null, but in new version returned null. My hypothesis is that in new version used new version of mtproto and it gets other data in packets, am i wrong?

@lmaxyz
Copy link

lmaxyz commented Mar 4, 2023

What user.username contains if you have several usernames?

@KyMaP13
Copy link
Contributor Author

KyMaP13 commented Mar 5, 2023

What user.username contains if you have several usernames?

Now, its None

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants