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

IIS HTTP/2 headers only response with END_STREAM flag on headers #47183

Open
1 task done
JamesNK opened this issue Mar 14, 2023 · 1 comment
Open
1 task done

IIS HTTP/2 headers only response with END_STREAM flag on headers #47183

JamesNK opened this issue Mar 14, 2023 · 1 comment
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions investigate
Milestone

Comments

@JamesNK
Copy link
Member

JamesNK commented Mar 14, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

A gRPC user is hosting their web app with IIS. They have noticed that a headers-only gRPC response (over HTTP/2) sends a HEADERS frame and an empty DATA frame with the END_STREAM flag. This goes against the gRPC HTTP/2 spec, which says the HEADERS frame should have the DATA frame.

Expected Behavior

IIS sends a HEADERS-only response with the END_STREAM flag in the HEADERS frame.

Note that this Kestrel has the correct behavior.

Steps To Reproduce

No response

Exceptions (if any)

No response

.NET Version

No response

Anything else?

No response

@ghost
Copy link

ghost commented Mar 15, 2023

To learn more about what this message means, what to expect next, and how this issue will be handled you can read our Triage Process document.
We're moving this issue to the .NET 8 Planning milestone for future evaluation / consideration. Because it's not immediately obvious what is causing this behavior, we would like to keep this around to collect more feedback, which can later help us determine how to handle this. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact work.

@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 25, 2023
@dotnet-policy-service dotnet-policy-service bot added the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Feb 6, 2024
@wtgodbe wtgodbe removed the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Feb 6, 2024
@dotnet-policy-service dotnet-policy-service bot added the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Feb 6, 2024
@wtgodbe wtgodbe removed the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Feb 13, 2024
@dotnet dotnet deleted a comment from dotnet-policy-service bot Feb 13, 2024
@dotnet dotnet deleted a comment from dotnet-policy-service bot Feb 13, 2024
@amcasey amcasey modified the milestones: .NET 8 Planning, Backlog Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions investigate
Projects
None yet
Development

No branches or pull requests

6 participants
@adityamandaleeka @JamesNK @amcasey @wtgodbe @mkArtakMSFT and others