-
Notifications
You must be signed in to change notification settings - Fork 82
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
Upgrade to Express 4 #261
Comments
I'm +1. Why not? On Fri, May 9, 2014 at 10:53 PM, Jay Harris notifications@github.com
|
So far, I haven't seen any disadvantages. There'll be a few more libraries included, but only because Express broke out the extras (like It should not break any existing My biggest concern is with Testem, since that is still using Express 3. |
Do you have a PR open for all this express/server/sockets work yet? |
I spent a good portion of time Monday and yesterday on Sockets (#239). I want to make sure that one is solid before Express 4, since I think the change from My goal is Sockets today, Express tomorrow. |
+10 for me. That would be lovely :) |
Websocket support would be awesome. We started using websockets recently and the lack of proxy support in lineman instantly became a major obstacle. Really made us see the value of server proxying :) |
Will this upgrade fix this output that I see when lineman runs?
If not, I'll open a separate issue for that. |
Yes, it will. |
Hey, what's the status here? @jayharris |
Ping @jayharris, did you ever try this out in a local development branch? |
I need to catch up with @jasonkarns about this. I have this functional, except for Push and for Partial Proxy. |
Is this anywhere near completion? |
Any objections to upgrading to Express 4? I've been working on a revised server component that includes WebSocket Proxy support, but it's built on Express 4.
The text was updated successfully, but these errors were encountered: