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

Already on GitHub? Sign in to your account

Add colon to match common logger spec #621

Merged
merged 1 commit into from Nov 19, 2013

Conversation

Projects
None yet
2 participants
Contributor

palexander commented Nov 18, 2013

Apache Common Logger uses a colon between the date and time, see http://httpd.apache.org/docs/1.3/logs.html#common and http://en.wikipedia.org/wiki/Common_Log_Format for examples. The lack of common breaks parsers that expect it (awstats is an example of this).

Member

spastorino commented Nov 19, 2013

Can you squash your commits into one?

Contributor

palexander commented Nov 19, 2013

Squashed

@spastorino spastorino added a commit that referenced this pull request Nov 19, 2013

@spastorino spastorino Merge pull request #621 from palexander/patch-1
Add colon to match common logger spec
38d788d

@spastorino spastorino merged commit 38d788d into rack:master Nov 19, 2013

1 check passed

default The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment