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

test 1184 fails in version 7.79.0 #7725

Closed
RiderALT opened this issue Sep 15, 2021 · 1 comment
Closed

test 1184 fails in version 7.79.0 #7725

RiderALT opened this issue Sep 15, 2021 · 1 comment
Labels

Comments

@RiderALT
Copy link

@RiderALT RiderALT commented Sep 15, 2021

full log attached.

10:46:51.248531 == Info: Issue another request to this URL: 'http://127.0.0.1.1184:36377/we/want/that/page/11840002'
10:46:51.248654 == Info: Hostname 127.0.0.1 was found in DNS cache
10:46:51.248701 == Info: Trying 127.0.0.1:33317...
10:46:51.248816 == Info: Connected to 127.0.0.1 (127.0.0.1) port 33317 (#1)
10:46:51.248837 == Info: allocate connect buffer
10:46:51.248867 == Info: Establish HTTP proxy tunnel to 127.0.0.1.1184:36377
10:46:51.248917 => Send header, 124 bytes (0x7c)
0000: CONNECT 127.0.0.1.1184:36377 HTTP/1.1
0027: Host: 127.0.0.1.1184:36377
0043: User-Agent: curl/7.79.0
005c: Proxy-Connection: Keep-Alive
007a:
10:46:51.255086 <= Recv header, 33 bytes (0x21)
0000: HTTP/1.1 200 Mighty fine indeed
10:46:51.255192 <= Recv header, 26 bytes (0x1a)
0000: Server: test tunnel 2000
10:46:51.255219 <= Recv header, 2 bytes (0x2)
0000:
10:46:51.255237 == Info: Proxy replied 200 to CONNECT request
10:46:51.255250 == Info: CONNECT phase completed
10:46:51.255310 => Send header, 109 bytes (0x6d)
0000: GET /we/want/that/page/11840002 HTTP/1.1
002a: Host: 127.0.0.1.1184:36377
0046: Accept: /
0053: User-Agent: 1184-agent
006b:
10:46:52.261405 == Info: Empty reply from server
10:46:52.261538 == Info: Closing connection 1
=== End of file trace1184

trace1184.txt

@bagder
Copy link
Member

@bagder bagder commented Sep 15, 2021

That test seems to be flaky for some reason.

@bagder bagder added the tests label Sep 15, 2021
bagder added a commit that referenced this issue Sep 16, 2021
The test should be fine and it works for me repeated when run manually,
but clearly it causes CI failures and it needs more research.

Reported-by: RiderALT on github
Fixes #7725
@bagder bagder closed this in 5b7aaef Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants