request.post() should raise HTTPRequestEntityTooLarge same as request.read()

[samuelcolvin]

request.read()

aiohttp/aiohttp/web_request.py

Lines 526 to 528 in 0bc7d52

	if self._client_max_size \
	and len(body) >= self._client_max_size:
	raise HTTPRequestEntityTooLarge

request.post()

aiohttp/aiohttp/web_request.py

Lines 583 to 585 in 0bc7d52

	if 0 < max_size < size:
	raise ValueError(
	'Maximum request body size exceeded')

Seems inconsistent to me that the two methods behave differently. Request.post() should raise HTTPRequestEntityTooLarge too.

Or am I missing something?

[asvetlov]

Agree.
Perhaps it is a migration artifact.
@samuelcolvin would you provide a fix?

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a [new issue] for related bugs.
If you feel like there's important points made in this discussion, please include those exceprts into that [new issue].
[new issue]: https://github.com/aio-libs/aiohttp/issues/new