Prefer static call to validate callback and use more precise exception

FriendsOfSymfony / FOSJsRoutingBundle

A pretty nice way to expose your Symfony routing to client applications.

1.48k stars 260 forks source link

Closed VincentLanglet closed 1 year ago

VincentLanglet commented 1 year ago

Hi @tobias-93,

I just noticed that

JsonpCallbackValidator::validate is static so doesn't require to instanciate a class
BadRequestHttpException should be preferred over HttpException for a 400 status

So I made the PR :)

tobias-93 commented 1 year ago

Thanks @VincentLanglet