[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

Update of patch #5913 (project freeciv): Status: Ready For Test => Done Assigned to:None => cazfi Open/Closed:Open => Closed

[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

Follow-up Comment #9, patch #5913 (project freeciv): Ok. This is a good improvement. Feel free to commit it. ___ Reply to this item at: ___ Message sent via/by G

[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

Follow-up Comment #8, patch #5913 (project freeciv): > I just tested current Freeciv-web on github with this patch I have fixed some other bugs on freeciv-proxy, but I have not pushed the protocol change. It must go in at the same time for both ends of the communication.

[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

Follow-up Comment #7, patch #5913 (project freeciv): I just tested current Freeciv-web on github with this patch, and it didn't work, so further changes are necessary. ___ Reply to this item at:

[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

Follow-up Comment #6, patch #5913 (project freeciv): Sure! Let's just make sure that Freeciv-web works also after this change. ___ Reply to this item at: ___ Mes

[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

Follow-up Comment #5, patch #5913 (project freeciv): > protocol is not symmetric in a sense that server->proxy has > different packet structure to what proxy->server has Meaning of header 'length' field differs between these. Proxy sends just the payload size while all the other parties (freeciv

[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

Follow-up Comment #4, patch #5913 (project freeciv): I suggest you add some debugging and look at the actual packets sent between Freeciv-proxy and the Freeciv server before and after the applying patch. > The patch includes some changes that were necessary for handling packets sent by the server

[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

Follow-up Comment #3, patch #5913 (project freeciv): It then seems that freeciv-web protocol is not symmetric in a sense that server->proxy has different packet structure to what proxy->server has. The patch includes some changes that were necessary for handling packets sent by the server (like in

[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

Follow-up Comment #2, patch #5913 (project freeciv): I don't fully undestand the patch, but it could be that the patch is correct, but that it requires some changes in Freeciv-proxy. The patch needs more testing. ___ Reply to this item at:

[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

Follow-up Comment #1, patch #5913 (project freeciv): I just tested this patch by applying it to the latest version of Freeciv-web from github. After applying the patch, no packets could be sent to or from the Freeciv server, so it is not possible start any games when the patch is applied. Please

[Freeciv-Dev] [patch #5913] json protocol: Handle "multiple packets in buffer" -case

URL: Summary: json protocol: Handle "multiple packets in buffer" -case Project: Freeciv Submitted by: cazfi Submitted on: Sat 07 Mar 2015 11:07:58 AM EET Category: freeciv-web P