Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(logs) separating access logs for admin and proxy reqs #1782

Merged
merged 1 commit into from
Nov 8, 2016
Merged

feat(logs) separating access logs for admin and proxy reqs #1782

merged 1 commit into from
Nov 8, 2016

Conversation

subnetmarco
Copy link
Member

This PR separates logs of the proxy and admin server, so that the two logs don't mix with each other.

Full changelog

  • Separating NGINX access logs of proxy and admin.

@subnetmarco subnetmarco added this to the 0.10 RC milestone Oct 28, 2016
@thibaultcha thibaultcha added pr/ready (but hold merge) No more concerns, but do not merge yet (probably a conflict of interest with another PR or release) and removed pr/status/needs review labels Oct 28, 2016
@subnetmarco subnetmarco merged commit b7049a1 into next Nov 8, 2016
@subnetmarco subnetmarco deleted the feat/logs branch November 8, 2016 21:14
cknowles pushed a commit to cknowles/docker-kong that referenced this pull request Mar 4, 2017
@c-knowles
0.10 separates access logs for admin and proxy
e8a696d 
Ideally we'd want to tag them somehow before feeding them to Docker but
for now at least the logs can get to Docker. Ref
Kong/kong#1782
cknowles pushed a commit to cknowles/docker-kong that referenced this pull request Mar 16, 2017
@c-knowles
0.10 separates access logs for admin and proxy
83bf499 
Ideally we'd want to tag them somehow before feeding them to Docker but
for now at least the logs can get to Docker. Ref
Kong/kong#1782
cknowles pushed a commit to cknowles/docker-kong that referenced this pull request May 17, 2017
@c-knowles
0.10 separates access logs for admin and proxy
782e9bb 
Ideally we'd want to tag them somehow before feeding them to Docker but
for now at least the logs can get to Docker. Ref
Kong/kong#1782
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thibaultcha thibaultcha thibaultcha approved these changes

Assignees
No one assigned
Labels
pr/ready (but hold merge) No more concerns, but do not merge yet (probably a conflict of interest with another PR or release)
Projects
None yet
Milestone
0.10 RC
Development

Successfully merging this pull request may close these issues.
2 participants
@subnetmarco @thibaultcha