Re: Downsizing CommonLogger

[Joshua Peek <josh@joshpeek.com>]

+1

I'll pull it in soon.

On Thu, May 14, 2009 at 12:14 AM, Matt Todd <chiology@gmail.com> wrote:
> +1
>
> On Wed, May 13, 2009 at 9:52 PM, Jeremy Kemper <jeremy@bitsweat.net> wrote:
>>
>> On Wed, Apr 29, 2009 at 8:21 AM, Michael Fellinger
>> <m.fellinger@gmail.com> wrote:
>> >
>> > On Wed, 29 Apr 2009 08:03:09 -0700
>> > Jeremy Kemper <jeremy@bitsweat.net> wrote:
>> >
>> >> CommonLogger calculates the content length itself. This duplicates
>> >> work done by the app or ContentLength middleware.
>> >>
>> >> The middleware is simpler, faster, and generates less garbage per
>> >> request by using the Content-Length header instead.
>> >>
>> >> Patch:
>> >>
>> >> http://github.com/jeremy/rack/commit/f9d38d9d866060be39192675adcc2efc19164c6f
>> >>
>> >> Best,
>> >> jeremy
>> >
>> > +1
>> > I've also long wanted to extract the format string like you did... so +2
>> > actually :)
>> >
>> > --
>> > ^ manveru
>> >
>>
>> This change is still on deck. Attention upstream :)
>>
>> jeremy
>
>
>
> --
> Matt Todd
> Highgroove Studios
> www.highgroove.com
> cell: 404-314-2612
> blog: maraby.org
>
> Scout - Web Monitoring and Reporting Software
> www.scoutapp.com
>



-- 
Joshua Peek