Re: imap.el workaround for Exchange 2007

[Reiner Steib <reinersteib+gmane@imap.cc>]

On Thu, Dec 18 2008, Dave Love wrote:

> Although this Microsoft product is apparently at least a year past its
> sell-by date, we just got it, and it broke my mail reading (which was OK
> with the one with a sell-by date of 2003).
>
> The first problem was fixed by `imap-enable-exchange-bug-workaround'.
> However, you're only likely to find that variable when debugging the
> breakage.  Why doesn't it use `*' generally -- as that seems to be
> equivalent -- and remove the need for users to find that variable?

You meant "*:*" (i.e. the variant used if
`imap-enable-exchange-bug-workaround' is t)?  If I recall the
discussion correctly, this is much slower.
Cf. <http://thread.gmane.org/gmane.emacs.gnus.general/66635>.

> Alternatively, perhaps it should check for Exchange and set it
> (per-server?) automatically, 

This has been discussed, but nobody implemented it yet.  I also think
this would be the right thing.

> or otherwise at least provide a user variable to customize.

Making `imap-enable-exchange-bug-workaround' customizable?

> The next problem (of many, doubtless...) is an assertion failure "In
> imap-parse-body".  I get a response like this for Exchange's version of
> article
> <URL:nntp://news.gmane.org/gmane.comp.clustering.gridengine.users/13804>,
> which I get direct from the gridengine list:
>
>
>   * 161 FETCH (UID 4779 RFC822.SIZE 35507 BODY ("multipart" "signed"
>   ("boundary" "----=_Part_2607_14582877.1229556098849") NIL NIL NIL
>   -1) BODY[HEADER.FIELDS (Subject From Date Message-Id References
>   In-Reply-To Xref X-Spam-Status X-Bogosity X-CRM114-Status To
>   Newsgroups)] {725}
[...]
>   )230 OK FETCH completed.
>
> Note the `-1' where there should be an unsigned integer, says rfc 3501:
>
>   body-fields     = body-fld-param SP body-fld-id SP body-fld-desc SP
>                     body-fld-enc SP body-fld-octets
>   body-fld-octets = number
>   number          = 1*DIGIT
>                       ; Unsigned 32-bit integer
>
> The following patch against current CVS fixes it for me (see the last
> hunk).  I also fixed some doc/commentary, and I assume the KTH mail
> address is obsolete.
>
> 2008-12-18  Dave Love  <fx@gnu.org>
>
> 	* imap.el: Fix author email.  Doc fixes.
> 	(imap-parse-body): Work around assertion failure in bogus Exchange 2007
> 	reply.
>
> --- imap.el	22 Jun 2008 13:24:10 +0100	7.47
> +++ imap.el	18 Dec 2008 14:10:53 +0000	
[... fixes for docs and comments stripped ]
> @@ -2862,7 +2871,12 @@
>  	;; as the standard says.
>  	(push (or (imap-parse-nstring) "7BIT") body) ;; body-fld-enc
>  	(imap-forward)
> -	(push (imap-parse-number) body)	;; body-fld-octets
> +	;; Exchange 2007 can return -1, contrary to the spec...
> +	(if (eq (char-after) ?-)
> +	    (progn
> +	      (skip-chars-forward "-0-9")
> +	      (push nil body))
> +	  (push (imap-parse-number) body)) ;; body-fld-octets

Installed.  (IIRC you still have write access to the repository, don't
you?)

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/