-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
HTTP2 Pusher interface is broken #65
Conversation
This pull request fixes 1 alert when merging 91aebaa into 1945d0d - view on LGTM.com fixed alerts:
|
91aebaa
to
bd31937
Compare
This pull request fixes 1 alert when merging bd31937 into 1945d0d - view on LGTM.com fixed alerts:
|
049ace2
to
6f030e4
Compare
73ae33f
to
cea31d3
Compare
92f923c
to
8ab5458
Compare
8ab5458
to
b9ed588
Compare
1a09133
to
ae71dcd
Compare
@@ -0,0 +1,70 @@ | |||
# TESTING PURPOSES ONLY |
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.
code smell, should be only one file
@@ -0,0 +1,4 @@ | |||
FROM nginx:1.19.5-alpine |
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.
code smell, should be configured in github actions
@@ -0,0 +1,31 @@ | |||
map $http_upgrade $connection_upgrade { |
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.
code smell, should be only one file
e69b3ab
to
e4c7a2b
Compare
e4c7a2b
to
f70fffd
Compare
Ref: #33