-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[webpack-dev-server and webpack-dev-middleware] Enable use other than express #4741
Comments
+1 on this, would improve startup times alot and the middleware apis are very similar if i recall. @alexander-akait - thoughts? |
We do have similar plans as mentioned in #4303. However, this is not yet prioritized. We will gladly accept a PR. |
@ScriptedAlchemy Fully agree, but we should start with https://github.com/webpack/webpack-dev-middleware, now we have compatibility using |
@alexander-akait Is there a specific reason to keep support for Express? I don't see the point of supporting Express.js since it might create unnecessary abstraction and maintenance burdens. |
@anonrig for dev server we don't need it, but webpack-dev-middleware should, because it is used for custom dev servers and we should support as much as possible, otherwise some developers will be angry 😄 |
Personally, I prefer to just use node core |
@silverwind And you're right too, I think the best solution would be an abstract API, it takes time, but allow to use very fast and any other solutions. Somebody want to try it (I will help), I personally think this is the right direction. |
Fixed:
|
Now we have tests only for express and connect I will add |
In the next major release we will migrate from |
Also will be great to have basic support of |
Fixed - #5275, also we have two examples - https://github.com/webpack/webpack-dev-server/tree/master/examples/app, anyway we supports any |
Feature Proposal
I recommend migrating to Fastify rather than using Express. The performance impact will be 7x (according to fastify readme), and I think it would be a lot beneficial for everybody.
Feature Use Case
I didn't understand why I need to write my system information while proposing a feature, but...
The text was updated successfully, but these errors were encountered: