-
-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UserAgent triggering false new device notifications #40
Comments
I noticed that behavior too, I think the only other way is using cookies and we know that lately this could create some issues with cookie laws. Same thing for dynamic IP address. |
Agreed, a lot of NewDevice Notifications is being created by this behavior. Configurable ignoring browser version may help. |
Author does not answer issue for a long time, I also do not observe activity in repository last time. Can I consider this package seriously for new application or mb better choose something else? |
Same problem, a lot of log notifications with this The script tried to call a method on an incomplete object. Please ensure that the class definition "Rappasoft\LaravelAuthenticationLog\Notifications\NewDevice" of the object you are trying to operate on was loaded before unserialize() gets called or provide an autoloader to load the class definition {"exception":"[object] (Error(code: 0): The script tried to call a method on an incomplete object. Please ensure that the class definition "Rappasoft\LaravelAuthenticationLog\Notifications\NewDevice" of the object you are trying to operate on was loaded before unserialize() gets called or provide an autoloader to load the class definition at /home/forge/app.unancor.com/vendor/laravel/framework/src/Illuminate/Notifications/SendQueuedNotifications.php:125) Any suggests? |
Same problem, a lot of log notifications with this I used muti Gurad auth Any suggests? |
mail is not sending when i login from new device of failed to login |
This packages works great, just I'm experiencing a few "false" new device notifications.
My question is should a user get a notification if they have updated their browser?
Take these two UserAgent strings:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.1 Safari/605.1.15
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.1.2 Safari/605.1.15
I'd suggest the version should be ignored?
The text was updated successfully, but these errors were encountered: