-
Notifications
You must be signed in to change notification settings - Fork 71
Open
Description
in method RedisStorage::getUnserializedValue, it should be possible to handle possible errors without crashing the whole app
return @unserialize($stored[1]); // intentionally @
could be encapsuled in try {} catch (Throwable $t) {} - to catch it and log it at least in php7 (7.1 alpha 3 +)
we are currently dealing with error "Invalid serialization data for DateTime object", and want to discuss possible/best solutions for this, so we cant fork it then, test in production and send pull request
thanks
(only way to reproduce it we have come up with is this:
php
Metadata
Metadata
Assignees
Labels
No labels