Skip to content

Change the AccessLogValve pattern#84

Merged
nebhale merged 1 commit intocloudfoundry:masterfrom
cgfrost:75588342-logging-pattern
Aug 22, 2014
Merged

Change the AccessLogValve pattern#84
nebhale merged 1 commit intocloudfoundry:masterfrom
cgfrost:75588342-logging-pattern

Conversation

@cgfrost
Copy link
Contributor

@cgfrost cgfrost commented Aug 8, 2014

The existing pattern doesn't show the IP address of the original requester.
This change adds a value of '%{org.apache.catalina.AccessLog.RemoteAddr}r' to
show the IP address of the original requester as provided by the RemoteIpValve.
'%a' will continue to show the IP address of the Router.

See: #75

[#75588342]

@cfdreddbot
Copy link

Hey cgfrost!

Thanks for submitting this pull request!

All pull request authors must have a Contributor License Agreement (CLA) on-file with us. Please sign the appropriate CLA (individual or corporate).

When sending signed CLA please provide your github username in case of individual CLA or the list of github usernames that can make pull requests on behalf of your organization.

The existing pattern doesn't show the IP address of the original requester.
This change adds a value of '%{org.apache.catalina.AccessLog.RemoteAddr}r' to
show the IP address of the original requester as provided by the RemoteIpValve.
'%a' will continue to show the IP address of the Router.

[#75588342]
@nebhale nebhale merged commit 5b922cd into cloudfoundry:master Aug 22, 2014
@cgfrost cgfrost deleted the 75588342-logging-pattern branch August 22, 2014 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants