Skip to content

Commit 0803b82

Browse files
committed
Changed tracking of logiin attempt to only track the attempt, not the result
1 parent 254b56c commit 0803b82

File tree

1 file changed

+3
-13
lines changed

1 file changed

+3
-13
lines changed

src/Listeners/LaravelMixpanelEventHandler.php

Lines changed: 3 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -11,23 +11,13 @@ class LaravelMixpanelEventHandler
1111
{
1212
public function onUserLoginAttempt($event)
1313
{
14-
$email = $event->credentials['email'] ?? '';
15-
$password = $event->credentials['password'] ?? '';
14+
$email = $event->credentials['email'] ?? $event['email'] ?? '';
1615

17-
if (starts_with(app()->version(), '5.1.')) {
18-
$email = $event['email'] ?? '';
19-
$password = $event['password'] ?? '';
20-
}
21-
22-
$authModel = config('auth.providers.users.model') ?? config('auth.model');
16+
$authModel = config('auth.providers.users.model', config('auth.model'));
2317
$user = app($authModel)
2418
->where('email', $email)
2519
->first();
26-
$eventName = 'Login Attempt Succeeded';
27-
28-
if ($user && ! auth()->validate($event->credentials)) {
29-
$eventName = 'Login Attempt Failed';
30-
}
20+
$eventName = 'Login Attempted';
3121

3222
event(new MixpanelEvent($user, $eventName));
3323
}

0 commit comments

Comments
 (0)