Re: Downsizing CommonLogger

[Matt Todd <chiology@gmail.com>]

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

+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

[-- Attachment #2: Type: text/html, Size: 1861 bytes --]