Gnus development mailing list
 help / color / mirror / Atom feed
From: Katsumi Yamaoka <yamaoka@jpl.org>
Subject: Re: rfc2231-parse-string: Decode encoded value after concatenating segments
Date: Thu, 28 Jul 2005 11:05:55 +0900	[thread overview]
Message-ID: <b9yslxzu1ks.fsf@jpl.org> (raw)
In-Reply-To: <v9ll3tiia5.fsf@marauder.physik.uni-ulm.de>

[-- Attachment #1: Type: text/plain, Size: 741 bytes --]

>>>>> In <v9ll3tiia5.fsf@marauder.physik.uni-ulm.de> Reiner Steib wrote:

> this change...

> ,----
>| 2004-12-21  Katsumi Yamaoka  <yamaoka@jpl.org>
>| 
>| 	* rfc2231.el (rfc2231-parse-string): Decode encoded value after
>| 	concatenating segments rather than before concatenating them.
>| 	Suggested by ARISAWA Akihiro <ari@mbf.ocn.ne.jp>.
> `----

> ... breaks displaying of external body parts for me if the file name
> spans more than one line, like this one:

Thanks for pointing it out.  I've fixed it in the v5-10 branch
and the trunk.  The change I'd made at the end of last year was
insufficient; I should have taken care that not only a segmented
parameter but also other parameters might be there.  Here's a
report I sent then:


[-- Attachment #2: Type: message/rfc822, Size: 895 bytes --]

From: Katsumi Yamaoka <yamaoka@jpl.org>
To: bugs@gnus.org
Cc: semi-gnus-ja@meadowy.org
Subject: Fixed: rfc2231.el decoder bug
Date: Tue, 21 Dec 2004 18:38:15 +0900
Message-ID: <b9yu0qgt2ig.fsf_-_@jpl.org>

Hi,

ARISAWA Akihiro reported there is a bug in the rfc2231 decoder.
He wrote that the decoder doesn't decode encoded long values
correctly as follows:

(rfc2231-parse-string
 (concat "Content-Disposition: attachment;"
	 (rfc2231-encode-string
	  "filename"
	  "一二三四五六七八九零一二三四五六七八九零.pdf")))
("content-disposition:attachment"
 (filename . "一二三四五六七八九零一二三四五六七八6eNm^[(B.pdf"))

I've fixed it in both the trunk and the v5-10 branch.

Regards,

      reply	other threads:[~2005-07-28  2:05 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-07-26 17:33 Reiner Steib
2005-07-28  2:05 ` Katsumi Yamaoka [this message]

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=b9yslxzu1ks.fsf@jpl.org \
    --to=yamaoka@jpl.org \
    /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).