-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
feat: http2 support for connect
compatibility application
#5267
Conversation
68eefa4
to
56a6dca
Compare
56a6dca
to
ff7f35a
Compare
All works fine except a deprecation warning - pillarjs/finalhandler#45, we can fix it here, I want to make a small patch and when it will be resolved we remove it |
@@ -2946,6 +2969,8 @@ class Server { | |||
*/ | |||
const allHeaders = []; | |||
|
|||
allHeaders.push({ key: "X_TEST", value: "TEST" }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just wondered after upgrading to webpack-dev-server
5.2.1 where this header is coming from and could track it down to this PR. Looks like it's there for quite some time already and I simply didn't notice until now. Does this serve any purpose or is it maybe a left-over from some testing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we I don't see it when merging, let's fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For Bugs and Features; did you add new tests?
Yes
Motivation / Use-Case
Allow to use
http2
withconnect
,express
still doesn't support http2Breaking Changes
No
Additional Info
No