Multi-client server, protocol versioning, streaming decompression #11

2026-07-16T13:59:48+02:00

TapTap commented

2026-07-16 13:59:48 +02:00

Three improvements:

1. Multi-client TCP server (transport_tcp.c, server.c)

server_listen now loops on accept() and forks a child per connection
Parent reaps children automatically via SIGCHLD=SIG_IGN
Signal handlers for SIGINT/SIGTERM clean up the listening socket on shutdown
Tested with parallel clients — both complete successfully

2. Protocol version negotiation (config.h, config.c, client_cli.c)

Added PROTOCOL_VERSION constant ("1.0.0")
Server checks client version on connect; mismatch sends STATUS_ERROR and exits
Client uses the constant via PROTOCOL_VERSION instead of a hardcoded string
Prevents silent corruption from mismatched client/server builds

3. Streaming decompression & error handling (compression.c, server.c, multiprocessing.c, file.c)

data_decompress now handles ZSTD_CONTENTSIZE_UNKNOWN by growing the output buffer dynamically (2x growth)
data_compress returns NULL instead of calling exit() on failure
data_decompress returns NULL instead of calling exit() on failure
Server receive path checks for NULL from decompression and sends STATUS_ERROR gracefully
Send path (file_send_single_calls) exits with a clear log message on compression failure

Three improvements: **1. Multi-client TCP server** (transport_tcp.c, server.c) - `server_listen` now loops on accept() and forks a child per connection - Parent reaps children automatically via SIGCHLD=SIG_IGN - Signal handlers for SIGINT/SIGTERM clean up the listening socket on shutdown - Tested with parallel clients — both complete successfully **2. Protocol version negotiation** (config.h, config.c, client_cli.c) - Added PROTOCOL_VERSION constant ("1.0.0") - Server checks client version on connect; mismatch sends STATUS_ERROR and exits - Client uses the constant via PROTOCOL_VERSION instead of a hardcoded string - Prevents silent corruption from mismatched client/server builds **3. Streaming decompression & error handling** (compression.c, server.c, multiprocessing.c, file.c) - data_decompress now handles ZSTD_CONTENTSIZE_UNKNOWN by growing the output buffer dynamically (2x growth) - data_compress returns NULL instead of calling exit() on failure - data_decompress returns NULL instead of calling exit() on failure - Server receive path checks for NULL from decompression and sends STATUS_ERROR gracefully - Send path (file_send_single_calls) exits with a clear log message on compression failure

TapTap added 1 commit 2026-07-16 13:59:49 +02:00

Multi-client server, protocol versioning, streaming decompression fixes efb98d8964

TapTap added 1 commit 2026-07-16 15:08:59 +02:00

Merge main into server-improvements: resolve conflicts (protocol signatures, error returns) a62894be05

TapTap merged commit c2436bef59 into main

2026-07-16 15:15:20 +02:00

TapTap deleted branch server-improvements

2026-07-16 15:15:23 +02:00

TapTap referenced this issue from a commit

2026-09-17 19:33:19 +02:00

fix(parity): empty-dir delete, per-dir filter errors, -R protect, stats parser

Sign in to join this conversation.