Re: [Tools-team] [codesprints] Datatracker Patch Report

Glen Barney <glen@amsl.com> Tue, 09 November 2010 03:30 UTC

Return-Path: <glen@amsl.com>
X-Original-To: tools-team@core3.amsl.com
Delivered-To: tools-team@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 8216628C1B8; Mon, 8 Nov 2010 19:30:24 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.599
X-Spam-Level:
X-Spam-Status: No, score=-102.599 tagged_above=-999 required=5 tests=[AWL=0.000, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id LszW7SSOXmNX; Mon, 8 Nov 2010 19:30:23 -0800 (PST)
Received: from mail.amsl.com (mail.amsl.com [64.170.98.20]) by core3.amsl.com (Postfix) with ESMTP id CF2E928C1A5; Mon, 8 Nov 2010 19:30:22 -0800 (PST)
Received: from localhost (localhost [127.0.0.1]) by c1a.amsl.com (Postfix) with ESMTP id 11D5CE08C2; Mon, 8 Nov 2010 19:30:46 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
Received: from c1a.amsl.com ([127.0.0.1]) by localhost (c1a.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id DW4rx0v36FFY; Mon, 8 Nov 2010 19:30:45 -0800 (PST)
Received: from [192.168.1.100] (173-8-133-91-SFBA.hfc.comcastbusiness.net [173.8.133.91]) by c1a.amsl.com (Postfix) with ESMTPSA id A6D27E076A; Mon, 8 Nov 2010 19:30:45 -0800 (PST)
Message-ID: <4CD8C05C.5090407@amsl.com>
Date: Mon, 08 Nov 2010 19:30:36 -0800
From: Glen Barney <glen@amsl.com>
Organization: AMS
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2.12) Gecko/20101027 Lightning/1.0b2 Thunderbird/3.1.6
MIME-Version: 1.0
To: Tony Hansen <tony@att.com>
References: <4CD8B280.9060404@amsl.com> <4CD8BA99.3040907@att.com>
In-Reply-To: <4CD8BA99.3040907@att.com>
X-Enigmail-Version: 1.1.1
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
Cc: tools-team@ietf.org, Russ Housley <housley@vigilsec.com>, Henrik Levkowetz <henrik@levkowetz.com>, "codesprints@ietf.org" <codesprints@ietf.org>, tools-development@ietf.org
Subject: Re: [Tools-team] [codesprints] Datatracker Patch Report
X-BeenThere: tools-team@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "The purpose of the TOOLS team is to provide IETF feedback and guidance during the development of software tools to support various parts of IETF activities." <tools-team.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/tools-team>, <mailto:tools-team-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/tools-team>
List-Post: <mailto:tools-team@ietf.org>
List-Help: <mailto:tools-team-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tools-team>, <mailto:tools-team-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 09 Nov 2010 03:30:24 -0000

On 11/8/2010 7:06 PM, Tony Hansen wrote:
> Your fix appears totally correct. Nit: I'm wondering if a better fix
> would be
>     elif 'multipart/' in content_type.lower()

I like that one better, yes.  I'll let Henrik figure out what he wants
to do with that.  :-)  Now that it's working, I don't want to touch it.  :-)

> Thanks for tracking the problem down Glen.
>     Tony

And thank you for the check!  :-)

Glen