Re: [Patch] Improve low-level network library, POP3 responsiveness on error

Hi all,

On 2018.01.20 15:22, Peter Bloomfield wrote:
On 01/20/2018 10:11:19 AM Sat, Albrecht Dreß wrote:
Hi all,
attached is a patch which improves the low-level network library, and should improve the responsiveness of POP3 connections on error.

For the network library, it introduces two new methods for shutting down the network connection, and for checking the connection status. The unit tests have been fixed, and the testing prerequisite ncat has been replaced by a trivial Python script.

For POP3 connections, on error the connection is now terminated without sending the QUIT command, as to avoid a “hanging” connection while waiting for a reply or a timeout to it.

As the patch modifies, I guess the meson build stuff needs to be modified as well, but unfortunately I don't know how (probably just removing the “sed_program” and “ncat_program” checks?). It would be great if someone with deeper insight could have a look at it.


Thanks for the patch! Looks like a really useful modification to the shutdown sequence. Given the frequency of errors with certain POP3 servers, it should make a marked improvement.

Pushed to master, with a companion patch to make the corresponding changes in the meson build files.

Thanks for this. The meson build worked for me with no problem, and the patch means the fetch dialog doesn't hang around for minutes due to strange Yahoo server errors/timeouts/whatever. (or at least it happens much less often in the few fetches I've done so far.)


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]