httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Brian Rectanus" <brect...@gmail.com>
Subject Re: [PATCH 43415] Logging remote port.
Date Thu, 20 Sep 2007 17:41:19 GMT
On 9/19/07, Plüm, Rüdiger, VF-Group <ruediger.pluem@vodafone.com> wrote:
>
>
> > -----Ursprüngliche Nachricht-----
> > Von: Adam Hasselbalch Hansen
> > Gesendet: Mittwoch, 19. September 2007 11:13
> > An: dev@httpd.apache.org
> > Betreff: Re: [PATCH 43415] Logging remote port.
> >
> >
> > Plüm wrote:
> >
> > > 1. Please provide a patch against trunk.
> >
> > mod_config_logger.c hasn't changed in trunk, so the patch
> > will work fine.
>
> Sorry. Just a default comment if someone sends in a patch that is not
> against trunk :-).
>
> >
> > > 2. Please also add a patch for the documentation.
> >
> > Done.
>
> Fine, I have already seen it in the report.
>
> >
> > > 3. I am not too happy with using %R, but to be honest I
> > have no better proposal :-).
> > >    Maybe other have.
> >
> > Well, then... ;)
>
> I will leave around for just one or two days. If nobody has a better
> idea we just take %R. Feel free to bug me if your patch falls off my radar.

There is an article on ONLamp a while back that used %S.  But I like
%R better ;)

http://www.onlamp.com/pub/a/apache/2004/04/22/blackbox_logs.html?page=3

The patch just uses apr_itoa()

-B

Mime
View raw message