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

AsyncStreamHandlerTest.asyncOptionsTest() is flaky #1735

Closed
TomGranot opened this issue Sep 28, 2020 · 1 comment
Closed

AsyncStreamHandlerTest.asyncOptionsTest() is flaky #1735

TomGranot opened this issue Sep 28, 2020 · 1 comment

Comments

@TomGranot
Copy link
Contributor

Following #1728 (comment), it seems that AsyncStreamHandlerTest.asyncOptionsTest() is flaky - there are a few different servers behind http://apache.org, and each returns either 4 or 5 methods based on the server we happened upon, crashing the test:

tom@Toms-MacBook-Pro async-http-client-fork % curl -i -X OPTIONS http://www.apache.org/
HTTP/1.1 200 OK
Date: Mon, 28 Sep 2020 11:24:45 GMT
Server: Apache/2.4.18 (Ubuntu)
Allow: POST,OPTIONS,GET,HEAD
Cache-Control: max-age=3600
Expires: Mon, 28 Sep 2020 12:24:45 GMT
Content-Length: 0
Content-Type: text/html

tom@Toms-MacBook-Pro async-http-client-fork % curl -i -X OPTIONS http://www.apache.org/
HTTP/1.1 200 OK
Date: Mon, 28 Sep 2020 11:53:21 GMT
Server: Apache
Allow: GET,POST,OPTIONS,HEAD,TRACE
Cache-Control: max-age=3600
Expires: Mon, 28 Sep 2020 12:53:21 GMT
Content-Length: 0
Content-Type: text/html

I need to rewrite so it does not rely on http://apache.org.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant