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 run in Au test tls_tunnel causing exception in proxy_protocol_client.py #9550

Closed
ywkaras opened this issue Mar 22, 2023 · 3 comments · Fixed by #11415
Closed

Test run in Au test tls_tunnel causing exception in proxy_protocol_client.py #9550

ywkaras opened this issue Mar 22, 2023 · 3 comments · Fixed by #11415
Assignees

Comments

@ywkaras
Copy link
Contributor

ywkaras commented Mar 22, 2023

Specifically this test run:

tr = Test.AddTestRun("test proxy_protocol_port - not in connect_ports")

Causes this exception:

2023-03-22 02:41:41,351 - root - INFO - Sending PROXY protocol version 2
Traceback (most recent call last):
  File "/home/wkaras/REPOS/TS/tests/_sandbox/tls_tunnel/proxy_protocol_client.py", line 176, in <module>
    sys.exit(main())
  File "/home/wkaras/REPOS/TS/tests/_sandbox/tls_tunnel/proxy_protocol_client.py", line 165, in main
    with context.wrap_socket(sock, server_hostname=args.sni) as ssock:
  File "/opt/y/1.0/lib/python3.10/ssl.py", line 513, in wrap_socket
    return self.sslsocket_class._create(
  File "/opt/y/1.0/lib/python3.10/ssl.py", line 1071, in _create
    self.do_handshake()
  File "/opt/y/1.0/lib/python3.10/ssl.py", line 1342, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLEOFError: EOF occurred in violation of protocol (_ssl.c:997)

Was this intended?

@ywkaras
Copy link
Contributor Author

ywkaras commented Aug 15, 2023

Still seeing this problem.

@masaori335
Copy link
Contributor

Recently, I faced below error on PRB. Is this related?

Run: Proxy Protocol v2 without blind tunneling on TLS connection to origin: Failed
...
Process: Default: Failed
       Setting up : Copying '/home/jenkins/workspace/Github_Builds/autest/src/build/proxy-verifier-v2.10.1/linux-amd64' to '/tmp/sandbox/proxy_protocol/pp-out-client-5/bin' - Passed
       Setting up : Copying 'replay/proxy_protocol_out.replay.yaml' to '/tmp/sandbox/proxy_protocol/pp-out-client-5/proxy_protocol_out.replay.yaml' - Passed
       Setting up : Copying '/home/jenkins/workspace/Github_Builds/autest/src/tests/gold_tests/autest-site/../../tools/proxy-verifier/ssl/client.pem' to '/tmp/sandbox/proxy_protocol/pp-out-client-5/client.pem' - Passed
       Setting up : Copying '/home/jenkins/workspace/Github_Builds/autest/src/tests/gold_tests/autest-site/../../tools/proxy-verifier/ssl/ca.pem' to '/tmp/sandbox/proxy_protocol/pp-out-client-5/ca.pem' - Passed
       Test : Checking that ReturnCode == 0 - Failed
          Reason: Returned Value 1 != 0
       Time-Out : Process finishes within expected time - Passed
          Reason: Returned value: 5.022040843963623 < 600.0
       file /tmp/sandbox/proxy_protocol/_output/7-tr-Default/stream.all.txt : Verify the client got a 200 response. - Failed
          Reason: Contents of /tmp/sandbox/proxy_protocol/_output/7-tr-Default/stream.all.txt did not contains expression: "HTTP/1.1 200 OK"
       file /tmp/sandbox/proxy_protocol/_output/7-tr-Default/stream.stdout.txt : There should be no Proxy Verifier violation errors. - Passed
          Reason: Contents of /tmp/sandbox/proxy_protocol/_output/7-tr-Default/stream.stdout.txt excludes expression

@maskit
Copy link
Member

maskit commented Jun 4, 2024

@masaori335 I can't tell if it's related without seeing the actual error log, but this issue is going to be closed by #11415. Create a new issue if you still see the error you mentioned.

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

Successfully merging a pull request may close this issue.

4 participants