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

Jump to bottom

x/net/http2: h2spec violation generic 2/2 & 2/3 #26326

Closed
fraenkel opened this issue Jul 11, 2018 · 4 comments
Closed

x/net/http2: h2spec violation generic 2/2 & 2/3 #26326

fraenkel opened this issue Jul 11, 2018 · 4 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone
Go1.12

Comments

@fraenkel
Copy link
Contributor 

Generic tests for HTTP/2 server
  2. Streams and Multiplexing
    × 2: Sends a WINDOW_UPDATE frame on half-closed (remote) stream
      -> The endpoint MUST accept WINDOW_UPDATE frame.
         Expected: DATA frame
           Actual: HEADERS Frame (length:28, flags:0x05, stream_id:1)
    × 3: Sends a PRIORITY frame on half-closed (remote) stream
      -> The endpoint MUST accept PRIORITY frame.
         Expected: DATA frame
           Actual: HEADERS Frame (length:28, flags:0x05, stream_id:1)
@fraenkel fraenkel mentioned this issue Jul 11, 2018
Closed
@ianlancetaylor ianlancetaylor added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Jul 11, 2018
@ianlancetaylor ianlancetaylor added this to the Go1.12 milestone Jul 11, 2018
@bradfitz
Copy link
Contributor

Can we get repro details on these bugs too? What command did you run and against which code?

@fraenkel
Copy link
Contributor Author

The command is $ ./h2spec -S -k -t -h 127.0.0.1 -p 8080 generic/2
I think its related to the "server" change I made in my set up which no longer sends any data back.
It expects a Data frame which assumes the server would send something back on a GET /.

If I change the server back to send back data, we pass. Which I assume is what this test assumes but the h2spec suite does not define how the Server should behave.

@fraenkel
Copy link
Contributor Author

I would lean toward closing this since its working properly with a response.

@fraenkel
Copy link
Contributor Author

Confirmed that the server is expected to send back some response therefore the test case does pass.

@golang golang locked and limited conversation to collaborators Jul 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Milestone
Go1.12
Development

No branches or pull requests
4 participants
@bradfitz @fraenkel @ianlancetaylor @gopherbot