Gnus development mailing list
 help / color / mirror / Atom feed
From: Gaute B Strokkenes <gs234@cam.ac.uk>
Subject: Re: nnimap versus tls
Date: Thu, 10 Jul 2003 12:42:54 +0100	[thread overview]
Message-ID: <87brw2sj41.fsf@cam.ac.uk> (raw)
In-Reply-To: <ilu1xwylt2g.fsf@latte.josefsson.org> (Simon Josefsson's message of "Thu, 10 Jul 2003 09:49:43 +0200")

On 10 jul 2003, jas@extundo.com wrote:

> Gaute B Strokkenes <gs234@cam.ac.uk> writes:
>
>> Debugger entered--Lisp error: (error "Selecting deleted buffer")
>> imap-arrival-filter(#<process imap<1>> "*** Received corrupted
>> data(-9) - server has terminated the connection abnormally\n")
>> delete-process(#<process imap<2>>) imap-close(#<buffer *imap
>> source*>)
>>
>> See?  gnutls-cli likes to spit out that line "*** Received
>> corrupted"... after the connection is closed.
>
> IMHO, gnutls-cli should print this to stderr instead of stdout.

It does.  It doesn't make a difference: the elisp manual states that
stderr and stdout of subprocesses can't be separated without using
shell redirection or something similar, and unless I misread the code,
we do no such thing.

>> Similarly openssl seems to like to write out "read:errno=0" when
>> the connection goes down.
>
> To stdout?  When -quiet is used?  Then I'd consider this a bug too,
> it should go to stderr.

It does go to stderr, but as above it makes no difference.

> Of course, nnimap should not misbehave even when the tunnel
> application print junk to stderr,

I'd like to point out that this is not the only problem case that
these patches correct.  There was a race in imap-wait-for-tag, in
which if the process dies we would not necessarily process everything
it output.  This lead to a similar error, but with the tail end of the
imapd's BYE response prepended to "*** Receved...".  I fixed that by
adding a final call to accept-process output, called only when
imap-wait-for-tag ends its loop due to the process exiting.  However,
that didn't the problem of the server outputting extra
junk--imap-wait-for-tag can return as soon as it gets all of the
response to the LOGOUT command, leaving the junk unprocessed (if the
timing is right.)

Looking at imap.el, there are lots of places where 'run and 'close
seem to be used synonymously as process statuses.  I don't think
that's quite correct--a network connection will remain open until
you've read everything the other end has sent you, but a subprocess
will not necessarily stay alive until you've read everything it has to
say.  (Or at least, that's the best estimation of the semantics that I
can make based on my observations; the emacs lisp manual is annoyingly
vague on this particular topic.)

> so your work and patches are still useful to have.  Thanks.

-- 
Gaute Strokkenes                        http://www.srcf.ucam.org/~gs234/
You mean you don't want to watch WRESTLING from ATLANTA?



  reply	other threads:[~2003-07-10 11:42 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-05-19 13:32 Gaute B Strokkenes
2003-07-02 23:46 ` Gaute B Strokkenes
2003-07-04 18:24 ` Kai Großjohann
2003-07-05  8:44   ` Gaute B Strokkenes
2003-07-05 12:53     ` Kai Großjohann
2003-07-06 12:56       ` Gaute B Strokkenes
2003-07-06 15:54         ` Kai Großjohann
2003-07-06 21:53           ` Gaute B Strokkenes
2003-07-07  6:08             ` Kai Großjohann
2003-07-07  7:26               ` Gaute B Strokkenes
2003-07-07  9:46                 ` Gaute B Strokkenes
2003-07-07 15:35                   ` Kai Großjohann
2003-07-08 22:20                     ` Gaute B Strokkenes
2003-07-09 19:46                       ` Kai Großjohann
2003-07-10  0:03                         ` Gaute B Strokkenes
2003-07-09 19:52                       ` Kai Großjohann
2003-07-10  1:25                         ` Gaute B Strokkenes
2003-07-10  7:24                           ` Kai Großjohann
2003-07-10 10:32                             ` Gaute B Strokkenes
2003-07-10  7:49                       ` Simon Josefsson
2003-07-10 11:42                         ` Gaute B Strokkenes [this message]
2003-07-10 12:13                           ` Simon Josefsson
2003-07-10 13:16                             ` Matthias Andree
2003-07-07 15:38                   ` Kai Großjohann
2003-07-07 19:37                     ` Gaute B Strokkenes

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87brw2sj41.fsf@cam.ac.uk \
    --to=gs234@cam.ac.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).