Consider throwing a MethodNotAllowedHttpException instead of BadRequestHttpException for Controller::requirePostRequest

[msbit]

For craft\web\Controller::requirePostRequest, as it's related specifically to the HTTP method expected, it seems to me a better match to throw a:

yii\web\MethodNotAllowedHttpException (resulting in a HTTP 405 error)

instead of the more general:

yii\web\BadRequestHttpException.

[brandonkelly]

Agree, that’s better. Just made that change for Craft 5 (77a88b3).