TCP Server refactoring #488

muellerr · 2021-09-28T15:11:21+02:00

muellerr commented

2021-09-28 15:11:21 +02:00

Fixes #470

This refactoring keeps the TCP connection opened until the client closes
it. It also increased the robustness of the TCP reception.

Because TCP is stream based and usually applied to newline separated
data, a special way to handle binary space packets is required.

The new SpacePacketParser class takes care of this by taking TC packet
IDs as optional start markers to parse for space packets in a given
buffer.

The refactored TCP server uses a ring buffer, a reception buffer and the
new parser to extract space packets from a stream in a safer and more robust way.
If the server is idle, it will use a configurable delay to decrease the CPU load.

A new reception mode state was introduced which could be used in the future to support newline separated data or other packet formats.

The changes were tested on Embedded Linux, Desktop Linux and Desktop Windows.

Unit tests for the new parser will follow in a separate PR

Fixes #470 This refactoring keeps the TCP connection opened until the client closes it. It also increased the robustness of the TCP reception. Because TCP is stream based and usually applied to newline separated data, a special way to handle binary space packets is required. The new SpacePacketParser class takes care of this by taking TC packet IDs as optional start markers to parse for space packets in a given buffer. The refactored TCP server uses a ring buffer, a reception buffer and the new parser to extract space packets from a stream in a safer and more robust way. If the server is idle, it will use a configurable delay to decrease the CPU load. A new reception mode state was introduced which could be used in the future to support newline separated data or other packet formats. The changes were tested on Embedded Linux, Desktop Linux and Desktop Windows. Unit tests for the new parser will follow in a separate PR

muellerr added 2 commits 2021-09-28 15:11:21 +02:00

TCP refactoring 09299802f0

This refactoring keeps the TCP connection opened until the client closes
it. It also increased the robustness of the TCP reception.

Because TCP is stream based and usually applied to newline separated
data, a special way to handle binary space packets is required.

The new SpacePacketParser class takes care of this by taking TC packet
IDs as as optional start markers to parse for space packets in a given
buffer.

The refactored TCP server uses a ring buffer, a reception buffer and the
new parser to extract space packets from a stream in a safer way.

wiretapping in runtime config now e536918804

muellerr changed title from ~~TCP refactoring~~ to WIP: TCP refactoring

2021-09-28 15:11:27 +02:00

muellerr added 1 commit 2021-09-28 15:13:04 +02:00

removed obsolete empty ctor bbea5e33bc

muellerr added 1 commit 2021-09-28 15:13:59 +02:00

cmake lists file update f02852d8d2

muellerr changed title from ~~WIP: TCP refactoring~~ to WIP: TCP Server refactoring

2021-09-28 15:14:48 +02:00

muellerr added 1 commit 2021-09-28 15:30:15 +02:00

documentation and bugfixes a4d6421510

muellerr added 1 commit 2021-09-28 15:33:39 +02:00