-
Notifications
You must be signed in to change notification settings - Fork 35
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
Middlewares in http module #47
Comments
Yeah, middleware makes sense and I don't see a problem with it. Still, it should be farily minimalist imo, just simple things like a generic server handler, or auth handler, or logging handler, etc. |
Indeed that is what I had on my mind. Circlet contains:
I would guess it is enough for the HTTP module too, and I can copy them verbatim from circlet if you agree or tell me which to copy. |
I have copied over and modified all of the above middleware, and added a parser for query-strings (that was built into circlet). |
I guess this new incarnation of Middleware is fine work. Thank you ❤️ |
Part of the circlet library was a small collection of middleware. We could add this to the HTTP module as well, if only as examples.
What do you think?
The text was updated successfully, but these errors were encountered: