Vasanko/sanic
2
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

More specific error messages on Payload Too Large.

Browse Source
This commit is contained in:
L. Kärkkäinen 2020-03-02 15:32:12 +02:00
parent 50cca39185
commit e73f26b0a3
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
sanic/http.py
View File

@ -116,7 +116,7 @@ class Http:
break break
await self._receive_more() await self._receive_more()
if pos >= self.request_max_size: if pos >= self.request_max_size:
raise PayloadTooLarge("Payload Too Large") raise PayloadTooLarge("Request header too large")
# Parse header content # Parse header content
try: try:
reqline, *raw_headers = buf[:pos].decode().split("\r\n") reqline, *raw_headers = buf[:pos].decode().split("\r\n")
@ -347,7 +347,7 @@ class Http:
# Request size limit # Request size limit
if self.request_bytes > self.request_max_size: if self.request_bytes > self.request_max_size:
self.keep_alive = False self.keep_alive = False
raise PayloadTooLarge("Payload Too Large") raise PayloadTooLarge("Request body too large")
# End of request body? # End of request body?
if not self.request_bytes_left: if not self.request_bytes_left:
self.request_body = None self.request_body = None

4
tests/test_payload_too_large.py
View File

@ -27,7 +27,7 @@ def test_payload_too_large_at_data_received_default(app):
response = app.test_client.get("/1", gather_request=False) response = app.test_client.get("/1", gather_request=False)
assert response.status == 413 assert response.status == 413
assert "Payload Too Large" in response.text assert "Request header" in response.text
def test_payload_too_large_at_on_header_default(app): def test_payload_too_large_at_on_header_default(app):
@ -40,4 +40,4 @@ def test_payload_too_large_at_on_header_default(app):
data = "a" * 1000 data = "a" * 1000
response = app.test_client.post("/1", gather_request=False, data=data) response = app.test_client.post("/1", gather_request=False, data=data)
assert response.status == 413 assert response.status == 413
assert "Payload Too Large" in response.text assert "Request body" in response.text