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

Log request headers, response headers and some body as warning on 500s #59

Merged
merged 5 commits into from
Sep 20, 2017
Merged

Log request headers, response headers and some body as warning on 500s #59

merged 5 commits into from
Sep 20, 2017

Commits on Sep 20, 2017

  1. Put user/orgID in request log if we have it

    @jml
    jml committed Sep 20, 2017
    Configuration menu
    Copy the full SHA
    36198be View commit details
    Browse the repository at this point in the history

  2. Show request & response headers in logs for 500s

    @jml
    jml committed Sep 20, 2017
    Configuration menu
    Copy the full SHA
    c65fbbe View commit details
    Browse the repository at this point in the history

  3. Only log 500s, not 400s 

    @bboreham was right
    @jml
    jml committed Sep 20, 2017
    Configuration menu
    Copy the full SHA
    4fc795c View commit details
    Browse the repository at this point in the history

  4. Don't log headers we don't have

    @jml
    jml committed Sep 20, 2017
    Configuration menu
    Copy the full SHA
    fbaac80 View commit details
    Browse the repository at this point in the history

  5. StatusOK if no explicit WriteHeader call

    @jml
    jml committed Sep 20, 2017
    Configuration menu
    Copy the full SHA
    542a545 View commit details
    Browse the repository at this point in the history