Skip to content

Possible fix of #49 - #103

Open
hookzof wants to merge 1 commit into
xelaj:mainfrom
hookzof:patch-1
Open

Possible fix of #49#103
hookzof wants to merge 1 commit into
xelaj:mainfrom
hookzof:patch-1

Conversation

@hookzof

@hookzof hookzof commented Jul 11, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@hookzof hookzof changed the title Possible fix of https://github.com/xelaj/mtproto/issues/49 Possible fix of #49 Jul 11, 2021
@quenbyako

Copy link
Copy Markdown
Member

@hookzof according to this description this pr won't work, unfortunately. I'm not sure, need to test it. But i appreciate your work, thanks.

shit, if telegram devs says that mtproto is opensource, why they didn't share simplest mtproto server, we could test it at least...

@quenbyako

Copy link
Copy Markdown
Member

won't close this pr, until test it

@AmarnathCJD

Copy link
Copy Markdown

works!
AmarnathCJD/gogram@15631a7

@quenbyako

Copy link
Copy Markdown
Member

@AmarnathCJD Could you please create Pull request with this chage? Just to have less differences between your fork and origin repo 馃槈

Also, Could you please create test case, what will happen, if provided prevID will be 1 billion for example?

@AmarnathCJD

AmarnathCJD commented Oct 21, 2022

Copy link
Copy Markdown

@AmarnathCJD Could you please create Pull request with this chage? Just to have less differences between your fork and origin repo wink

Also, Could you please create test case, what will happen, if provided prevID will be 1 billion for example?

The commit history is broken :( I will try

and if the prevID is 1 B, the GenerateMessageID runs until the value reaches greater than 1B, This solves the msg_seq_too_high (33) a far as i tested

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.

3 participants