|
34 | 34 | #include "src/stirling/source_connectors/socket_tracer/testing/container_images/node_12_3_1_container.h" |
35 | 35 | #include "src/stirling/source_connectors/socket_tracer/testing/container_images/node_14_18_1_alpine_container.h" |
36 | 36 | #include "src/stirling/source_connectors/socket_tracer/testing/container_images/node_client_container.h" |
37 | | -#include "src/stirling/source_connectors/socket_tracer/testing/container_images/python_3_10_container.h" |
| 37 | +#include "src/stirling/source_connectors/socket_tracer/testing/container_images/python_min_310_container.h" |
38 | 38 | #include "src/stirling/source_connectors/socket_tracer/testing/container_images/ruby_container.h" |
39 | 39 | #include "src/stirling/source_connectors/socket_tracer/testing/protocol_checkers.h" |
40 | 40 | #include "src/stirling/source_connectors/socket_tracer/testing/socket_trace_bpf_test_fixture.h" |
@@ -86,7 +86,9 @@ class Node14_18_1AlpineContainerWrapper |
86 | 86 | int32_t PID() const { return process_pid(); } |
87 | 87 | }; |
88 | 88 |
|
89 | | -class Python310ContainerWrapper : public ::px::stirling::testing::Python310Container { |
| 89 | +// Python 3.10 and later use SSL_write_ex and SSL_read_ex. This test case is itended to cover |
| 90 | +// this case. See https://github.com/pixie-io/pixie/issues/1113 for more details. |
| 91 | +class PythonMin310ContainerWrapper : public ::px::stirling::testing::PythonMin310Container { |
90 | 92 | public: |
91 | 93 | int32_t PID() const { return process_pid(); } |
92 | 94 | }; |
@@ -181,7 +183,7 @@ http::Record GetExpectedHTTPRecord() { |
181 | 183 |
|
182 | 184 | using OpenSSLServerImplementations = |
183 | 185 | Types<NginxOpenSSL_1_1_0_ContainerWrapper, NginxOpenSSL_1_1_1_ContainerWrapper, |
184 | | - NginxOpenSSL_3_0_8_ContainerWrapper, Python310ContainerWrapper, |
| 186 | + NginxOpenSSL_3_0_8_ContainerWrapper, PythonMin310ContainerWrapper, |
185 | 187 | Node12_3_1ContainerWrapper, Node14_18_1AlpineContainerWrapper>; |
186 | 188 |
|
187 | 189 | TYPED_TEST_SUITE(OpenSSLTraceTest, OpenSSLServerImplementations); |
|
0 commit comments