-
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
Replace white/blacklists with error hanlder and never re-raise errors #98
Replace white/blacklists with error hanlder and never re-raise errors #98
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
Do you have any feedback on this? @tfausak |
I'm sorry it took me so long to look at this. I like some of what you've done here. In particular I think sending the error to a method in Stoplight (via This feels like a good improvement on my work in #97. I want to think a little more about how to handle the fallback case without so much |
@tfausak Checkout this PR when you have a chance. Thanks!