]> arthur.barton.de Git - ngircd.git/commit
Revert "Increase read buffer size for server connections"
authorAlexander Barton <alex@barton.de>
Fri, 1 May 2020 20:38:41 +0000 (22:38 +0200)
committerAlexander Barton <alex@barton.de>
Sun, 3 May 2020 22:46:56 +0000 (00:46 +0200)
commit629a45ee0f4469729f6a9c22a983000fcc39fe61
tree36b5c3eeb24dffd24710907aa64fc9e09e2c1f35
parent8d414b079c1b4b0b8c88c8ab5cf4b27bf0942752
Revert "Increase read buffer size for server connections"

This reverts commit c6e3c13f27744971fcb1d2de4e561d3bcdaa5aed.

This sounded like the right approach at first, but I'm not that sure
that it really makes sense to have different sizes of read buffers: the
per-connection read buffer only needs to keep data that is needed to
parse one full command, be it plain text, encrypted and/or compressed.
Then ngIRCd should handle this one command, move leftover data to the
beginning of the buffer and read the next chunk from the network that is
missing to get the next complete command (512 bytes at max).

So I revert this for now and try to fix the logic in Read_Request(),
which is broken nevertheless, as it results in servers becoming
disconnected during "server burst" when "big" lists are transferred.
src/ngircd/conn.c
src/ngircd/defines.h