Skip to content

Quarantine ServerAcceptsRequestWithheaderTotalSizeWithinLimit#21191

Merged
1 commit merged into
masterfrom
rybrande/Quarnetine
Apr 25, 2020
Merged

Quarantine ServerAcceptsRequestWithheaderTotalSizeWithinLimit#21191
1 commit merged into
masterfrom
rybrande/Quarnetine

Conversation

@ryanbrandenburg
Copy link
Copy Markdown
Contributor

Assert.Equal() Failure
          ↓ (pos 0)
Expected: HTTP/1.1 200 OK\r\nDate: Fri, 18 Mar 7887 1···
Actual:   
          ↑ (pos 0)
------
   at Microsoft.AspNetCore.Testing.StreamBackedTestConnection.Receive(String[] lines) in /_/src/Servers/Kestrel/shared/test/StreamBackedTestConnection.cs:line 125
   at Microsoft.AspNetCore.Server.Kestrel.InMemory.FunctionalTests.RequestHeaderLimitsTests.ServerAcceptsRequestWithHeaderTotalSizeWithinLimit(Int32 headerCount, Int32 extraLimit) in /_/src/Servers/Kestrel/test/InMemory.FunctionalTests/RequestHeaderLimitsTests.cs:line 35
   at Microsoft.AspNetCore.Server.Kestrel.InMemory.FunctionalTests.RequestHeaderLimitsTests.ServerAcceptsRequestWithHeaderTotalSizeWithinLimit(Int32 headerCount, Int32 extraLimit) in /_/src/Servers/Kestrel/test/InMemory.FunctionalTests/RequestHeaderLimitsTests.cs:line 45
--- End of stack trace from previous location ---

@Pilchie
Copy link
Copy Markdown
Member

Pilchie commented Apr 24, 2020

👀

@Pilchie
Copy link
Copy Markdown
Member

Pilchie commented Apr 25, 2020

Reran the failed legs, and adding auto-merge

@ghost
Copy link
Copy Markdown

ghost commented Apr 25, 2020

Hello @Pilchie!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 2589a63 into master Apr 25, 2020
@ghost ghost deleted the rybrande/Quarnetine branch April 25, 2020 21:26
@halter73
Copy link
Copy Markdown
Member

This should be fixed by #21272.

@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Jun 6, 2023
This pull request was closed.
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants