Project

General

Profile

Bug #2228

%p in accesslog.format does not correctly split the port off ipv6 sockets

Added by jo.henke over 6 years ago. Updated about 6 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
Category:
mod_accesslog
Target version:
Start date:
2010-06-24
Due date:
% Done:

100%

Missing in 1.5.x:
No

Description

Hello,

I'm using "%p" in my 'accesslog.format' string to see, how many people actually use SSL. Since I bound lighttpd to a specific IPv6 address (via 'server.bind'), I find strings like the following in the %p column of my log:

cdef:6789::1:80

So it seems like the socket address is split at the first colon instead of the last one. A quick workaround would probably be to replace 'strchr' with 'strrchr' here:

http://redmine.lighttpd.net/projects/lighttpd/repository/revisions/2699/entry/branches/lighttpd-1.4.x/src/mod_accesslog.c#L848

Regards,
Jo.

Associated revisions

Revision 48f1cf79 (diff)
Added by stbuehler about 6 years ago

mod_accesslog: fix %p for ipv6 sockets (fixes #2228, thx jo.henke)

git-svn-id: svn://svn.lighttpd.net/lighttpd/branches/lighttpd-1.4.x@2734 152afb58-edef-0310-8abb-c4023f1b3aa9

Revision 2734 (diff)
Added by stbuehler about 6 years ago

mod_accesslog: fix %p for ipv6 sockets (fixes #2228, thx jo.henke)

History

#1 Updated by stbuehler about 6 years ago

  • Status changed from New to Fixed
  • % Done changed from 0 to 100

Applied in changeset r2734.

Also available in: Atom