Skip to content

HTTP/2: Fix flakey response rate timeout tests #32139

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.

Sign up for GitHub

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
Merged
merged 2 commits into from
Apr 27, 2021
Merged

HTTP/2: Fix flakey response rate timeout tests #32139

merged 2 commits into from
Apr 27, 2021

Conversation

@JamesNK
Copy link
Member

@JamesNK JamesNK commented Apr 24, 2021

Fixes #21520

@ghost ghost added the area-runtime label Apr 24, 2021
@JamesNK JamesNK mentioned this pull request Apr 24, 2021
Merged
halter73
halter73 approved these changes Apr 26, 2021
View reviewed changes
Copy link
Member

@halter73 halter73 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good find.

@JamesNK JamesNK merged commit 4a382c1 into main Apr 27, 2021
@JamesNK JamesNK deleted the jamesnk/http2-timeout-flakeytests branch April 27, 2021 00:35
@ghost ghost added this to the 6.0-preview5 milestone Apr 27, 2021
@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@halter73 halter73 halter73 approved these changes

@jkotalik jkotalik Awaiting requested review from jkotalik

@Tratcher Tratcher Awaiting requested review from Tratcher

Assignees

No one assigned

Labels

area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions

Projects

None yet

Milestone

6.0-preview5

Development

Successfully merging this pull request may close these issues.

[Servers] Investigate MissingAuthority_200Status failures

4 participants

@JamesNK @halter73 @amcasey