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

support JSON logging #1208

Merged
merged 6 commits into from Feb 28, 2017

Conversation

Projects
None yet
1 participant
@kazuho
Member

kazuho commented Feb 22, 2017

Adds escape attribute to the access-log configuration directives. The attribute takes one of the two values: apache, json. apache is the default.

If the value is json, the escape sequence changes from \xNN to \u00NN, and the special value used for logging missing values is changed from - to null.

Note: Regarding the escape sequence, the reason for using \u00NN is that it is impossible to assume that the input (e.g. URL) is UTF-8. Therefore, we treat it as ISO-8859-1 in order to preserve the original octets. Users with the knowledge of the application should convert the strings to appropriate encoding laterwards, if required.

@kazuho kazuho added this to the v2.2 milestone Feb 28, 2017

@kazuho kazuho merged commit bed0d5b into master Feb 28, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

kazuho added a commit that referenced this pull request Mar 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment