suth / laravel-sift Goto Github PK

The config file should determine if (and possibly which) auth events should be tracked by default.

I'd like to implement an easy way to queue requests to the SiftClient class. Laravel makes queuing easy but it might be even better to have something like a shouldQueue method.

The RecordLoginSuccess and RecordLoginFailure listeners don't have access to the session when using async queues, so they fail.

Hi there,

When user is trying to login with incorrect credentials the package is observing this with Suth\LaravelSift\Listeners\RecordLoginFailure

But $sift->getUserId($event->user) is causes an exception:

Call to a member function getAuthIdentifier() on null
in SiftScience.php (line 44)

This is because user is not authenticated at the point and $event->user == null

Please fix

Right now the Sift user ID the package uses for tracking purposes is the user's email address. There should be a way to customize this behavior.

I'm just wondering, is there any reason that the project is not using the 2.* SiftScience library? Is there any incompatibility that you may be aware of?

Thank you very much for your time and for the project.

Jan.