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
There was some discussion on this in #240 and I really strongly disagree with the idea of catching BaseException (or even Exception for that matter), and I'm not alone.
We've had to remove this check and re-implement our own, which is much the same as this package's check but doesn't catch the BaseException. There might be some other errors to catch, like #340 does for CacheBackend, but they really should be caught properly and not just caught via BaseException.
I must also say Thank You to everyone who maintains this package, it really is very helpful for us 😄
The text was updated successfully, but these errors were encountered:
There was some discussion on this in #240 and I really strongly disagree with the idea of catching
BaseException
(or evenException
for that matter), and I'm not alone.We've had to remove this check and re-implement our own, which is much the same as this package's check but doesn't catch the
BaseException
. There might be some other errors to catch, like #340 does forCacheBackend
, but they really should be caught properly and not just caught viaBaseException
.I must also say Thank You to everyone who maintains this package, it really is very helpful for us 😄
The text was updated successfully, but these errors were encountered: