-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Support middlewares #49
base: master
Are you sure you want to change the base?
Conversation
Nice contribution! I suggest you add global middleware support. For example for logging or metrics. To have something like this:
Also, some examples/docs would be nice. |
@lcd1232 Sorry for the code review. Thanks for suggesting the middleware implementation. As @kochetkov-av mentioned, I agree with the way he suggested. |
@osamingo I fixed all issues |
My suggestion is to add "after" middleware test if order testing, like this:
|
WHAT
Support middlewares
WHY
see #45