mtproto icon indicating copy to clipboard operation
mtproto copied to clipboard

Possible fix of #49

Open hookzof opened this issue 3 years ago • 5 comments

hookzof avatar Jul 11 '21 14:07 hookzof

@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 avatar Jul 12 '21 10:07 quenbyako

won't close this pr, until test it

quenbyako avatar Jul 12 '21 10:07 quenbyako

works! https://github.com/AmarnathCJD/gogram/commit/15631a7050c18bcd381fdb1686710f40208d766a

AmarnathCJD avatar Oct 21 '22 11:10 AmarnathCJD

@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?

quenbyako avatar Oct 21 '22 13:10 quenbyako

@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

AmarnathCJD avatar Oct 21 '22 13:10 AmarnathCJD