-
Notifications
You must be signed in to change notification settings - Fork 372
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
Won't show exception if cache driver is not supported #236
base: master
Are you sure you want to change the base?
Conversation
Instead of throwing an exception, the configuration file explicitly mentioned the unsupported drivers, the package should gracefully handle it instead of throwing an exception.
I am using Dynamo_DB in AWS Lambda Trows same Exception Error:
This Below is the Stack Trace: Not Complete as Includes Confidential Info
|
@huzaifaarain May I know What will happen Exactly in the background Just for My Knowledge Will we get the IP of User If exception is not shown as it is not supporting the tag cache? |
I experienced the issue and fixed in my fork: https://github.com/InteractionDesignFoundation/laravel-geoip/releases/tag/3.4.2 |
Instead of throwing an exception, the configuration file explicitly mentioned the unsupported drivers, the package should gracefully handle it instead of throwing an exception.