Skip to content

Reduce the log level of 'request limit exceeded' messages#2788

Merged
Kludex merged 2 commits intoKludex:mainfrom
shevron:feature/reduce-req-limit-exceeded-log-level
Feb 15, 2026
Merged

Reduce the log level of 'request limit exceeded' messages#2788
Kludex merged 2 commits intoKludex:mainfrom
shevron:feature/reduce-req-limit-exceeded-log-level

Conversation

@shevron
Copy link
Copy Markdown
Contributor

@shevron shevron commented Jan 12, 2026

Summary

This PR is a follow-up on #2782

In #2430 a WARNING level message was added when the defined --limit-max-requests limit is reached and the process is restarted.

IMHO this should be an INFO level message, as it doesn't indicate any abnormal or potentially broken behavior, but in fact a 100% normal behavior. Using WARNING level message really pollutes the logs in high traffic servers, and the only way to silence these messages is to filter them in the logger configuration based on the content of the message, which is far from ideal.

This PR only changes the level of the message from WARNING to INFO.

Checklist

  • I understand that this PR may be closed in case there was no previous discussion. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly (there doesn't seem to be much to update)

@Kludex Kludex enabled auto-merge (squash) February 15, 2026 18:34
@Kludex Kludex merged commit a03d9f6 into Kludex:main Feb 15, 2026
19 checks passed
@Kludex Kludex mentioned this pull request Feb 16, 2026
theseriff pushed a commit to theseriff/uvicorn that referenced this pull request Mar 9, 2026
Co-authored-by: Shahar Evron <shahar@DS-shahar-MBP.local>
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