Skip to content

Conversation

@mastercactapus
Copy link
Member

Description:
Fixes various incorrect status code responses

  • Hitting a limit (like too many unacked alerts) now correctly responds with 409 Conflict fixing a regression.
  • Body size limit errors now respond with 413 Content Too Large
  • Aborted/disconnected requests will be sent a 499 to distinguish from 400
  • Requests in the queue that timeout will now get 408 Request Timeout (as opposed to the 429 Too Many Requests when the queue is full)

Also a test was added for the body size limit, as prior to this change they would incorrectly result in 500 errors.

KatieMSB
KatieMSB previously approved these changes Aug 22, 2025
Co-authored-by: Cameron Hall <cjhall1283@gmail.com>
@mastercactapus mastercactapus merged commit e990fd4 into master Aug 22, 2025
8 checks passed
@mastercactapus mastercactapus deleted the fix-body-size-response branch August 22, 2025 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants