Re: [PATCH] download: Do not wrap TLS port on GnuTLS >= 3.7.7.

2022-08-04 Thread Maxime Devos
On 02-08-2022 09:59, Ludovic Courtès wrote: + (if (module-defined? (resolve-interface '(gnutls)) + 'set-session-record-port-close!) ;GnuTLS >= 3.7.7 resolve-module (and presumably also sets #:ensure #t by default, which sometimes causes 'module not found' messages

Re: bug#56867: [PATCH] download: Do not wrap TLS port on GnuTLS >= 3.7.7.

2022-08-04 Thread Chris Vine
On Thu, 04 Aug 2022 16:20:12 +0200 Ludovic Courtès wrote: > Ludovic Courtès skribis: > > > The custom input/output port wrapping the TLS session record port would > > introduce overhead, and it would also prevent its uses in a non-blocking > > context--e.g., with Fibers. The port close mechanis

Re: [bug#56867] [PATCH] download: Do not wrap TLS port on GnuTLS >= 3.7.7.

2022-08-04 Thread Thiago Jung Bauermann
Hello Ludo, I don't have any comment/insight on what you're doing in general, except about one of your points below: Ludovic Courtès writes: > First, I noticed that GnuTLS doesn’t implement ‘write_wait_fd’, only > ‘read_wait_fd’ (not sure how problematic that is): > > scheme@(guile-user)> ,us

Re: [bug#56867] [PATCH] download: Do not wrap TLS port on GnuTLS >= 3.7.7.

2022-08-04 Thread Ludovic Courtès
Hi, Thiago Jung Bauermann skribis: > Ludovic Courtès writes: > >> First, I noticed that GnuTLS doesn’t implement ‘write_wait_fd’, only >> ‘read_wait_fd’ (not sure how problematic that is): >> >> scheme@(guile-user)> ,use(web client) >> scheme@(guile-user)> (define p (open-socket-for-uri "https:

Re: bug#56867: [PATCH] download: Do not wrap TLS port on GnuTLS >= 3.7.7.

2022-08-04 Thread Ludovic Courtès
Ludovic Courtès skribis: > The custom input/output port wrapping the TLS session record port would > introduce overhead, and it would also prevent its uses in a non-blocking > context--e.g., with Fibers. The port close mechanism added in GnuTLS > 3.7.7 allows us to get rid of that wrapper. > > *