Re: [PATCH] Store remote host ip on commit

Sander Striker wrote:
>
> Hi,
>
> This patch adds functionality to subversion to obtain
> the ip of the host where a commit is originating from.
>
> Use 'svnlook ${REPOS} rev ${REV} remotehost' to see where
> a commit came from. 'remotehost' will not be set when
> it couldn't be determined (or when using ra_local).
>
> It was a question by Eric Gillespie on irc that got me
> to take a peek to see how much effort it was to implement.
> Turned out to be not much effort.
>
> Thoughts?

What about naming it client_host, commit_host or something like
that. The host always doesn't have to be remote, and then we
could also record that the commit was from 127.0.0.1 for ra_local
commits so the value is always filled.

For security and debugging purposes, it also may be interesting
to have the client send what it thinks its own IP address is, since
NAT will often times change the IP that the svn server sees.
I don't know if this is too useful.

Best,
Blair

-- 
Blair Zajac <blair@orcaware.com>
Web and OS performance plots - http://www.orcaware.com/orca/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org