Skip Menu |
 

This queue is for tickets about the Encode CPAN distribution.

Report information
The Basics
Id: 111419
Status: resolved
Priority: 0/
Queue: Encode

People
Owner: Nobody in particular
Requestors: florz [...] florz.de
Cc: pali [...] cpan.org
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



MIME-Version: 1.0
X-Spam-Status: No, score=-3.901 tagged_above=-99.9 required=10 tests=[AWL=-2.000, BAYES_00=-1.9, RP_MATCHES_RCVD=-0.001] autolearn=ham
Content-Disposition: inline
X-Spam-Flag: NO
content-type: text/plain; charset="utf-8"
Message-ID: <20160125164008.GA6823 [...] florz.florz.de>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
X-Spam-Score: -3.901
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id E5D022402C7 for <cpan-bug+Encode [...] hipster.bestpractical.com>; Mon, 25 Jan 2016 11:40:28 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ghCMH7D-TRaa for <cpan-bug+Encode [...] hipster.bestpractical.com>; Mon, 25 Jan 2016 11:40:27 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 691AF24027A for <bug-Encode [...] rt.cpan.org>; Mon, 25 Jan 2016 11:40:27 -0500 (EST)
Received: (qmail 10464 invoked by alias); 25 Jan 2016 16:40:26 -0000
Received: from rain.florz.de (HELO rain.florz.de) (62.216.164.86) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Mon, 25 Jan 2016 08:40:23 -0800
Received: from florz.florz.de ([192.168.0.121]:54524) by rain.florz.de with esmtps (TLSv1.2:DHE-RSA-AES256-SHA256:256) (Exim 4.80) (envelope-from <florz [...] florz.de>) id 1aNkBW-0004DG-Rf for bug-Encode [...] rt.cpan.org; Mon, 25 Jan 2016 17:40:10 +0100
Received: from florz by florz.florz.de with local (Exim 4.80) (envelope-from <florz [...] florz.de>) id 1aNkBU-0001n8-KX for bug-Encode [...] rt.cpan.org; Mon, 25 Jan 2016 17:40:08 +0100
Delivered-To: cpan-bug+Encode [...] hipster.bestpractical.com
User-Agent: Mutt/1.5.21 (2010-09-15)
Subject: incorrect unfolding and other decoding bugs
Return-Path: <florz [...] florz.de>
X-RT-Mail-Extension: encode
X-Original-To: cpan-bug+Encode [...] hipster.bestpractical.com
X-Spam-Check-BY: la.mx.develooper.com
Date: Mon, 25 Jan 2016 17:40:08 +0100
X-Spam-Level:
To: bug-Encode [...] rt.cpan.org
From: Florian Zumbiehl <florz [...] florz.de>
X-RT-Original-Encoding: ascii
X-RT-Interface: Email
Content-Length: 180
Download (untitled) / with headers
text/plain 180b
Bug #67569 has been marked as resolved even though the bug obviously hasn't been fixed. For explanation and patch, see there: https://rt.cpan.org/Public/Bug/Display.html?id=67569
MIME-Version: 1.0
In-Reply-To: <20160125164008.GA6823 [...] florz.florz.de>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <20160125164008.GA6823 [...] florz.florz.de>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-23866-1453743063-242.111419-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 682
Download (untitled) / with headers
text/plain 682b
On Mon Jan 25 11:40:29 2016, florz@florz.de wrote: Show quoted text
> Bug #67569 has been marked as resolved even though the bug obviously hasn't > been fixed. For explanation and patch, see there: > > https://rt.cpan.org/Public/Bug/Display.html?id=67569
I've added all test vectors in RFC2047, pp.11-12 https://github.com/dankogai/p5-encode/commit/b0d30fbf504b69caad5cc43b5c25c9a3b0166fa1 And Encode 2.80 passes all that. Plus so far as I see some of your test vectors are wrong. For instance "=?us-ascii?q?foo?=\r\n bar" => "foo bar" Should be "foobar", not "foo bar". That is a single line when decoded and in such cases the leading white spaces immediately after CRLF is ignored. Dan
MIME-Version: 1.0
X-Spam-Status: No, score=-4.901 tagged_above=-99.9 required=10 tests=[AWL=1.000, BAYES_00=-1.9, FROM_OUR_RT=-4, RP_MATCHES_RCVD=-0.001] autolearn=ham
In-Reply-To: <rt-4.0.18-23866-1453743063-646.111419-6-0 [...] rt.cpan.org>
Content-Disposition: inline
X-Spam-Flag: NO
X-RT-Interface: API
References: <RT-Ticket-111419 [...] rt.cpan.org> <20160125164008.GA6823 [...] florz.florz.de> <rt-4.0.18-23866-1453743063-646.111419-6-0 [...] rt.cpan.org>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <20160125175714.GM24603 [...] florz.florz.de>
content-type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -4.901
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 6D33F240380 for <cpan-bug+Encode [...] hipster.bestpractical.com>; Mon, 25 Jan 2016 12:57:27 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id mVttuzZKzIsH for <cpan-bug+Encode [...] hipster.bestpractical.com>; Mon, 25 Jan 2016 12:57:25 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 1F03A2402CB for <bug-Encode [...] rt.cpan.org>; Mon, 25 Jan 2016 12:57:24 -0500 (EST)
Received: (qmail 18782 invoked by alias); 25 Jan 2016 17:57:24 -0000
Received: from rain.florz.de (HELO rain.florz.de) (62.216.164.86) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Mon, 25 Jan 2016 09:57:21 -0800
Received: from florz.florz.de ([192.168.0.121]:54786) by rain.florz.de with esmtps (TLSv1.2:DHE-RSA-AES256-SHA256:256) (Exim 4.80) (envelope-from <florz [...] florz.de>) id 1aNlO8-00051Z-GJ for bug-Encode [...] rt.cpan.org; Mon, 25 Jan 2016 18:57:16 +0100
Received: from florz by florz.florz.de with local (Exim 4.80) (envelope-from <florz [...] florz.de>) id 1aNlO6-0002NC-AX for bug-Encode [...] rt.cpan.org; Mon, 25 Jan 2016 18:57:14 +0100
Delivered-To: cpan-bug+Encode [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #111419] incorrect unfolding and other decoding bugs
User-Agent: Mutt/1.5.21 (2010-09-15)
Return-Path: <florz [...] florz.de>
X-Spam-Check-BY: la.mx.develooper.com
X-Original-To: cpan-bug+Encode [...] hipster.bestpractical.com
X-RT-Mail-Extension: encode
Date: Mon, 25 Jan 2016 18:57:14 +0100
X-Spam-Level:
To: Dan Kogai via RT <bug-Encode [...] rt.cpan.org>
From: Florian Zumbiehl <florz [...] florz.de>
RT-Message-ID: <rt-4.0.18-4238-1453744648-961.111419-0-0 [...] rt.cpan.org>
Content-Length: 1146
Download (untitled) / with headers
text/plain 1.1k
Hi, Show quoted text
> Plus so far as I see some of your test vectors are wrong. For instance
Yes, I think some are, and as I wrote a long time ago, I'd be happy to fix those if there is any chance that the fix will actually get merged. Show quoted text
> "=?us-ascii?q?foo?=\r\n bar" => "foo bar" > > Should be "foobar", not "foo bar". That is a single line when decoded and in such cases the leading white spaces immediately after CRLF is ignored.
That one is actually correct. Could you point to where in the relevant RFCs you think the behavior you suggest is specified? You might want to start reading at sections 2.2.3 and 3.2.2 of RFC 5322, the former of which this quote is from: | The process of moving from this folded multiple-line representation | of a header field to its single line representation is called | "unfolding". Unfolding is accomplished by simply removing any CRLF | that is immediately followed by WSP. Each header field should be | treated in its unfolded form for further syntactic and semantic | evaluation. An unfolded header field has no length restriction and | therefore may be indeterminately long. Regards, Florian
MIME-Version: 1.0
X-Spam-Status: No, score=-5.234 tagged_above=-99.9 required=10 tests=[AWL=0.667, BAYES_00=-1.9, FROM_OUR_RT=-4, RP_MATCHES_RCVD=-0.001] autolearn=ham
In-Reply-To: <rt-4.0.18-23866-1453743063-646.111419-6-0 [...] rt.cpan.org>
Content-Disposition: inline
X-Spam-Flag: NO
X-RT-Interface: API
References: <RT-Ticket-111419 [...] rt.cpan.org> <20160125164008.GA6823 [...] florz.florz.de> <rt-4.0.18-23866-1453743063-646.111419-6-0 [...] rt.cpan.org>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <20160129015430.GY24603 [...] florz.florz.de>
content-type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -5.234
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 7489B2402D0 for <cpan-bug+Encode [...] hipster.bestpractical.com>; Thu, 28 Jan 2016 20:54:50 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id b2puqX-wBP-8 for <cpan-bug+Encode [...] hipster.bestpractical.com>; Thu, 28 Jan 2016 20:54:49 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id DE7962400B2 for <bug-Encode [...] rt.cpan.org>; Thu, 28 Jan 2016 20:54:48 -0500 (EST)
Received: (qmail 16640 invoked by alias); 29 Jan 2016 01:54:47 -0000
Received: from rain.florz.de (HELO rain.florz.de) (62.216.164.86) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Thu, 28 Jan 2016 17:54:38 -0800
Received: from florz.florz.de ([192.168.0.121]:38168) by rain.florz.de with esmtps (TLSv1.2:DHE-RSA-AES256-SHA256:256) (Exim 4.80) (envelope-from <florz [...] florz.de>) id 1aOyGe-00016Y-Lu for bug-Encode [...] rt.cpan.org; Fri, 29 Jan 2016 02:54:32 +0100
Received: from florz by florz.florz.de with local (Exim 4.80) (envelope-from <florz [...] florz.de>) id 1aOyGc-0000p8-DM for bug-Encode [...] rt.cpan.org; Fri, 29 Jan 2016 02:54:30 +0100
Delivered-To: cpan-bug+Encode [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #111419] incorrect unfolding and other decoding bugs
User-Agent: Mutt/1.5.21 (2010-09-15)
Return-Path: <florz [...] florz.de>
X-Spam-Check-BY: la.mx.develooper.com
X-Original-To: cpan-bug+Encode [...] hipster.bestpractical.com
X-RT-Mail-Extension: encode
Date: Fri, 29 Jan 2016 02:54:30 +0100
X-Spam-Level:
To: Dan Kogai via RT <bug-Encode [...] rt.cpan.org>
From: Florian Zumbiehl <florz [...] florz.de>
RT-Message-ID: <rt-4.0.18-11239-1454032491-1356.111419-0-0 [...] rt.cpan.org>
Content-Length: 433
Download (untitled) / with headers
text/plain 433b
Hi, BTW, I just checked, these are the two that arguably are wrong: "=?us-ascii?q?foo?==?us-ascii?q?bar?=" => "foo=?us-ascii?q?bar?=" "foo =?us-ascii?q?=20?==?us-ascii?q?bar?=" => "foo =?us-ascii?q?bar?=" Though it's not really clear whether they are indeed wrong, as RFC2047 is somewhat ambiguous there. All the other examples are definitely correct. So, is there any chance you will finally merge the fix? Regards, Florian
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-11239-1454032491-1356.111419-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <RT-Ticket-111419 [...] rt.cpan.org> <20160125164008.GA6823 [...] florz.florz.de> <rt-4.0.18-23866-1453743063-646.111419-6-0 [...] rt.cpan.org> <20160129015430.GY24603 [...] florz.florz.de> <rt-4.0.18-11239-1454032491-1356.111419-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-18621-1459276575-310.111419-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 56
I suppose that this bug is finnaly fixed in version 2.83
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-18621-1459276575-310.111419-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <RT-Ticket-111419 [...] rt.cpan.org> <20160125164008.GA6823 [...] florz.florz.de> <rt-4.0.18-23866-1453743063-646.111419-6-0 [...] rt.cpan.org> <20160129015430.GY24603 [...] florz.florz.de> <rt-4.0.18-11239-1454032491-1356.111419-0-0 [...] rt.cpan.org> <rt-4.0.18-18621-1459276575-310.111419-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-15630-1460637450-301.111419-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 101
Download (untitled) / with headers
text/plain 101b
On Tue Mar 29 14:36:15 2016, PALI wrote: Show quoted text
> I suppose that this bug is finnaly fixed in version 2.83


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.