middlewares / recaptcha Goto Github PK

Returning a 403 response if the validation does not pass looks a reasonable default to me, but it is not always what a user may desire.

Maybe he wants to set a flash message and redirect to an error page, or who knows what else.

It would be nice to make this configurable, allowing the middleware to receive a callable returning an instance of ResponseInterface:

class Recaptcha implements MiddlewareInterface
{
    public function __construct(callable $responseFactory = null)
    {
        if (null === $responseFactory) {
            $responseFactory = function () {return Utils\Factory::createResponse(403);}
        }
        $this->responseFactory = $responseFactory;
    }

    public function process(ServerRequestInterface $request, DelegateInterface $delegate)
    {
        if (!$this->isValid($request)) {
            return ($this->responseFactory)();
        }
    }
}

what do you say? Seems reasonable?

You have google/recaptcha as a dependency. I've recently updated this with a 1.2 release. It should be backwards compatible, but obviously you'd want to test that. Happy to respond to issues over at https://github.com/google/recaptcha