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" <debian-...@v.nix.is> 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 debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to