From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.0 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 33ED920229; Thu, 20 Oct 2016 20:50:52 +0000 (UTC) Date: Thu, 20 Oct 2016 20:50:52 +0000 From: Eric Wong To: Mishael A Sibiryakov Cc: unicorn-public@bogomips.org Subject: Re: [PATCH] Add some tolerance (RFC2616 sec. 19.3) Message-ID: <20161020205052.GA30292@starla> References: <1476954332.1736.156.camel@junki.org> <20161020175554.GA15564@starla> <1476995156.1736.166.camel@junki.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1476995156.1736.166.camel@junki.org> List-Id: Mishael A Sibiryakov wrote: > It's just a temporary fork and I'm too lazy )) > Whoops, my fault.  No worries :) > > > +  def test_multiline_header_0d0a > > > +    parser = HttpParser.new > > > +    parser.buf << "GET / HTTP/1.0\r\nX-Multiline-Header: foo > > > bar\r\n\tcha cha\r\n\tzha zha\r\n\r\n" > > > > I expect code to be wrapped at 80 lines or less.  Fixed locally. > > (I need big fonts, even 80 is a compromise, I really prefer 64) > > Line 221 with "ssl bullshit" guided me to this. Ah, that's an old test from Mongrel, probably not worth fixing; just trying to keep new code cleaner.