ArranJacques / laravel-aws-cognito-auth

An authentication driver for Laravel for authenticating users in AWS Cognito User Pools.
Other
74 stars 26 forks source link

Laravel 5.7 Compatibility #21

Closed moebrowne closed 4 years ago

moebrowne commented 5 years ago

As of the release of Laravel 5.7 the auth events must be passed the guard name.

This was added here: https://github.com/laravel/framework/commit/69cddedae349956c5f38455e861e3fc490d89a07 and the order of the arguments was changed in a later commit (https://github.com/laravel/framework/commit/f5d8c0a673aa9fc6cd94aa4858a0027fe550a22e).

I have added a requirement of upwards of laravel/framework 5.7 to the Composer JSON though I don't know how you want to handle tagging a change like this as it's technically a breaking change