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
fixed faulty message id #2040
fixed faulty message id #2040
Conversation
Hmm, are you sure we can do this? Looks a bit like a protocol change. I'll
take a better look later
…On Sun, Jan 19, 2020, 21:13 Chiller Dragon ***@***.***> wrote:
@ChillerDragon <https://github.com/ChillerDragon> requested your review
on: #2040 <#2040> fixed faulty message
id.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#2040?email_source=notifications&email_token=AADXYBEJTQ3S5QIJODT7KV3Q6SX4ZA5CNFSM4KI3JJU2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWB7SRNI#event-2961123509>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADXYBFTMGV7JBSFPYU6GV3Q6SX4ZANCNFSM4KI3JJUQ>
.
|
@Learath2 tbh I have no clue. But I tested it and the client can still connect to ddnet and non ddnet servers and works fine as far as I know. Also tested with dummy. But yes please take a look. Im scared. I am especially unsure about this part because I improvised there: |
Ok here is everything I tested (all worked fine) Client:
Server:
|
(cherry picked from commit teeworlds/teeworlds@9023796)
0d8622c
to
1b4131a
Compare
@heinrich5991 if you have a couple minutes can you take a look at this? I'm not completely familiar with the entire thing |
Looks good to me. Thanks. bors r+ |
2040: fixed faulty message id r=heinrich5991 a=ChillerDragon (cherry picked from commit teeworlds/teeworlds@9023796) Co-authored-by: oy <Tom_Adams@web.de>
Build failed
|
bors retry |
2040: fixed faulty message id r=heinrich5991 a=ChillerDragon (cherry picked from commit teeworlds/teeworlds@9023796) Co-authored-by: oy <Tom_Adams@web.de>
Build failed
|
bors retry |
2040: fixed faulty message id r=heinrich5991 a=ChillerDragon (cherry picked from commit teeworlds/teeworlds@9023796) Co-authored-by: oy <Tom_Adams@web.de>
Build failed
|
New day, lets hope Microsoft fixed their repository. bors retry |
2040: fixed faulty message id r=heinrich5991 a=ChillerDragon (cherry picked from commit teeworlds/teeworlds@9023796) Co-authored-by: oy <Tom_Adams@web.de>
Build succeeded
|
(cherry picked from commit teeworlds/teeworlds@9023796)