Mercurial > p > roundup > code
diff roundup/cgi/exceptions.py @ 8408:e882a5d52ae5
refactor: move RateLimitExceeded to roundup.cgi.exceptions
RateLimitExceeded is an HTTP exception that raises code 429. Move it
to roundup.cgi.exceptions where all the other exceptions that result
in http status codes are located. Also make it inherit from
HTTPException since it is one.
Also add docstrings for all HTTP exceptions and order HTTPExceptions
by status code.
BREAKING CHANGE: if somebody is importing RateLimitExceeded they will
need to change their import. I consider it unlikely anybody is using
RateLimitExceeded. Detectors and extensions are unlikely to raise
RateLimitExceeded. So I am leaving it out of the upgrading doc. Just
doc in change log.
| author | John Rouillard <rouilj@ieee.org> |
|---|---|
| date | Sun, 10 Aug 2025 21:27:06 -0400 |
| parents | 700424ba015c |
| children | ef1ea918b07a |
line wrap: on
line diff
--- a/roundup/cgi/exceptions.py Sun Aug 10 20:57:57 2025 -0400 +++ b/roundup/cgi/exceptions.py Sun Aug 10 21:27:06 2025 -0400 @@ -24,6 +24,11 @@ pass +class NotModified(HTTPException): + """HTTP 304 status code""" + pass + + class NotFound(HTTPException): """HTTP 404 status code unless self.response_code is set to 400 prior to raising exception. @@ -31,13 +36,13 @@ pass -class NotModified(HTTPException): - """HTTP 304 status code""" +class PreconditionFailed(HTTPException): + """HTTP 412 status code""" pass -class PreconditionFailed(HTTPException): - """HTTP 412 status code""" +class RateLimitExceeded(HTTPException): + """HTTP 429 error code""" pass