Skip to content
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

Methods around cookie parser are provided, but middleware isnt #83

Open
maxkorp opened this issue Sep 1, 2020 · 0 comments
Open

Methods around cookie parser are provided, but middleware isnt #83

maxkorp opened this issue Sep 1, 2020 · 0 comments

Comments

@maxkorp
Copy link
Contributor

maxkorp commented Sep 1, 2020

I can see how this may be intentional, but body parser is included by default, as are the middlewares for it (Express.Middleware.json for example), but while Request.cookies etc are exposed, there is no Express.Middleware.cookie. It's a super simple binding, and including it would require adding cookieparser to the deps, so I can see where you may not want that, but it's a pretty small package and I think the odds of use are high enough to add the convenience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant