You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If there's something wrong with Redis this will crash the app at boot. In our case Error undefined method smembers for RedisClient, but I think the specific error is less important than generally robust error handling.
I see this as a "nice to have" gem, so I'd rather it fail to record usage than crash at boot. IMO the whole internal Redis code in the gem should be wrapped in a rescue.
If there's something wrong with Redis this will crash the app at boot. In our case
Error undefined method smembers for RedisClient
, but I think the specific error is less important than generally robust error handling.I see this as a "nice to have" gem, so I'd rather it fail to record usage than crash at boot. IMO the whole internal Redis code in the gem should be wrapped in a
rescue
.The text was updated successfully, but these errors were encountered: