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

Reduce log level of 501 Not Implemented responses #321

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Reduce log level of 501 Not Implemented responses #321

wants to merge 1 commit into from

Commits on Jan 22, 2021

  1. Limit logging of 501 Not Implemented errors 

    For public facing websites it's not uncommon to have exploitation suites run
random strings as methods in small batches of requests. This commit channels
501s away from errors logs into info logs as we may still want to know that
unknown methods are being used, typically though it's a part of the normal
course of operation and don't want it to fill logging with high priority
messages and potentially fall through into alerting for some users.
    @tggreene
    tggreene committed Jan 22, 2021
    Configuration menu
    Copy the full SHA
    2648c1d View commit details
    Browse the repository at this point in the history