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.

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

Actually allow HTTP logging to be enabled #2373

Merged
merged 2 commits into from
Apr 21, 2015
Merged

Conversation

otoolep
Copy link
Contributor

@otoolep otoolep commented Apr 21, 2015

No description provided.

@otoolep otoolep force-pushed the control_http_loggign branch 2 times, most recently from ec7210a to 4914bde Compare April 21, 2015 21:59
@otoolep
Copy link
Contributor Author

otoolep commented Apr 21, 2015

HTTP access logging must be controlled at handler-construction time, since construction is when routing, and logging, are actually set.

@otoolep
Copy link
Contributor Author

otoolep commented Apr 21, 2015

Currently failing on Circle, investigating.

@otoolep otoolep force-pushed the control_http_loggign branch from 418514e to 5ff1bd1 Compare April 21, 2015 22:21
@mark-rushakoff
Copy link
Contributor

Confirmed HTTP logging visible again. 👍

@otoolep
Copy link
Contributor Author

otoolep commented Apr 21, 2015

Thanks @mark-rushakoff

otoolep added a commit that referenced this pull request Apr 21, 2015
Actually allow HTTP logging to be enabled
@otoolep otoolep merged commit b50c503 into master Apr 21, 2015
@otoolep otoolep deleted the control_http_loggign branch April 21, 2015 22:35
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.

2 participants