Welcome! Log In Create A New Profile

Advanced

Re: [PATCH 1 of 2] HTTP: Add client source port to any error that is logged

Quanah Gibson-Mount
April 24, 2014 07:38PM
--On April 24, 2014 at 10:26:07 PM +0400 Maxim Dounin <mdounin@mdounin.ru>
wrote:

>> Yes, that is true, but why only implement a partial solution? With CGN,
>> only logging the IP is fairly useless in all cases. To truly get useful
>> information going forward, the IP + PORT of the client should logged in
>> all cases.
>
> Access log certainly can be configured to provide enough
> enformation to match any given error log message to a port if
> needed. There is no need to implement anything, solution is
> already here.

The error log currently only provides the IP. While I'm guessing you could
do things like correlate timestamps, it's still going to be a pain. Having
the port readily available everywhere makes tracking a specific user much
easier to do.

> And, by asking about "why implement a partical solution" you are
> overlooking the fact that proposed solution is partial as well -
> it doesn't change c->addr_text to ensure proper logging in all
> places (this would be a bad idea for other reasons, but it's
> another question), but rather tries to hack on the http error
> logging code to introduce remote port logging. This is far from
> being a complete solution.

I'm certainly willing to address any deficiencies, but I'd want to make
sure it would follow whatever you want in the product before investing more
time on it. ;) For now it meets the needs of our customer in Belgium who
has to start dealing with the legal requirements of client port logging
sooner than later.

--Quanah

--
Quanah Gibson-Mount
Server Architect
Zimbra, Inc
--------------------
Zimbra :: the leader in open source messaging and collaboration

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH 0 of 2] TRAC 531 - Add source port logging for HTTP(S) error logging and all proxied email client connections

Quanah Gibson-Mount 1468 April 24, 2014 01:22PM

[PATCH 2 of 2] MAIL: Always log the source port of the client

Quanah Gibson-Mount 467 April 24, 2014 01:22PM

Re: [PATCH 2 of 2] MAIL: Always log the source port of the client

Maxim Dounin 439 April 24, 2014 01:48PM

Re: [PATCH 2 of 2] MAIL: Always log the source port of the client

Quanah Gibson-Mount 442 April 24, 2014 07:40PM

[PATCH 1 of 2] HTTP: Add client source port to any error that is logged

Quanah Gibson-Mount 524 April 24, 2014 01:22PM

Re: [PATCH 1 of 2] HTTP: Add client source port to any error that is logged

Maxim Dounin 468 April 24, 2014 01:38PM

Re: [PATCH 1 of 2] HTTP: Add client source port to any error that is logged

Quanah Gibson-Mount 462 April 24, 2014 01:42PM

Re: [PATCH 1 of 2] HTTP: Add client source port to any error that is logged

Quanah Gibson-Mount 466 April 24, 2014 01:46PM

Re: [PATCH 1 of 2] HTTP: Add client source port to any error that is logged

Maxim Dounin 479 April 24, 2014 01:58PM

Re: [PATCH 1 of 2] HTTP: Add client source port to any error that is logged

Quanah Gibson-Mount 449 April 24, 2014 02:08PM

Re: [PATCH 1 of 2] HTTP: Add client source port to any error that is logged

Maxim Dounin 439 April 24, 2014 02:28PM

Re: [PATCH 1 of 2] HTTP: Add client source port to any error that is logged

Quanah Gibson-Mount 547 April 24, 2014 07:38PM

Re: [PATCH 1 of 2] HTTP: Add client source port to any error that is logged

Maxim Dounin 485 April 25, 2014 07:50AM

Re: [PATCH 0 of 2] TRAC 531 - Add source port logging for HTTP(S) error logging and all proxied email client connections

itpp2012 810 May 23, 2014 02:56PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 166
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready