tags 569786 wontfix
stop
Dear v.nix.is,
I marked this request as wontfix, because implementing this seems to be
a duplicate effort, since you could just put
LogFormat "%v %v %h %l %u %t \"%r\" %>s %O \"%{Referer}i\" \"%{User-Agent}i\""
combined
into your apache config. That way you can put the second %v wherever
you want on the line and it will be written out.
So your use-case can be achieved with a bit of a configuration change,
therefore I don't see it appropriate to add yet another option.
The only issue I forsee, is that vlogger unfortunately uses other
data column numbers directly like this:
my $reqsize = $this_line[10];
or:
$reqsize = $this_line[1] + $this_line[2];
If someone would submit a patch that adds parameters to specify the
reqsize column and/or mod_logio columns, that would be OK with me.
But I don't know how many people are using the mysql statistics feature
anyways... Seems to be a big hack, to be honest.
If you don't agree with this wontfix, please reply to the bugreport and
point out your reasons.
Thank you for taking the time to report this,
Gergely
On Sun, 14 Feb 2010 09:59:48 +0000, "v.nix.is" <[email protected]> writes:
> Package: vlogger
> Version: 1.3-3
> Severity: wishlist
>
> I found that using vlogger with programs such as awstats that expect
> logs to be in an unaltered combined log format doesn't work. awstats
> for example will simply die on any line that doesn't include a host.
>
> So I made a small change to vlogger to add a -p option for
> "PassThrough mode". It's a very small patch and with it vlogger won't
> munge log lines at all before writing them to subdirectories as it
> does normally.
>
> The patch is here:
> http://github.com/avar/linode-etc/commit/94f0d7d22773d4430f1490ddea9858f2cbcd50e8
>
> I didn't modify the man page or other things in the package because
> this works for me and I wanted to see what the author thought of it.
>
> -- System Information:
> Debian Release: squeeze/sid
> APT prefers testing
> APT policy: (650, 'testing'), (500, 'unstable')
> Architecture: amd64 (x86_64)
>
> Kernel: Linux 2.6.32-x86_64-linode11 (SMP w/4 CPU cores)
> Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
> Shell: /bin/sh linked to /bin/dash
>
> Versions of packages vlogger depends on:
> ii libtimedate-perl 1.2000-1 collection of modules to
> manipulat
> ii perl 5.10.1-9 Larry Wall's Practical
> Extraction
>
> vlogger recommends no packages.
>
> Versions of packages vlogger suggests:
> ii libdbi-perl 1.609-1 Perl Database Interface (DBI)
>
> -- no debconf information
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
