-
Notifications
You must be signed in to change notification settings - Fork 40
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
Insure only rescuing safe errors #99
Insure only rescuing safe errors #99
Conversation
This will fix bolshakov#91. It still needs some tests.
…light about this error call `handler#handle(error)`.
…unding/stoplight into error-handler-module-triquels
Needs test refactor and fixing.
I still feel like this is too complicated. See 2a12612 for my approach. |
@tfausak I think your commit make sense, I'll feel good how it prevent users from easily shooting them selves in the foot. I also see the performance improvements with using a Proc that gets used over and over again instead instantiating a new object every time for the error handler. |
I appreciate the work that you put into this, and I'm glad that you helped me figure out the best way to solve this problem. That being said, I'm going to move forward with my changes in #97. |
Remove dynamic
Module.===
but insure only rescuing safe errorsNeeds test refactor and fixing.