Skip to content

Commit 806674a

Browse files
committed
fix(ci): add netty-tcnative to proxy test deps (Round 8)
Http2ProtocolProxyHandlerTest requires native OpenSSL for HTTP/2 over TLS. Round 4 removed netty-tcnative-boringssl-static to fix a Bazel cycle dependency, but tests still need it. Add it back only to test deps to avoid the cycle in production code.
1 parent 97fce71 commit 806674a

2 files changed

Lines changed: 5 additions & 0 deletions

File tree

WORKSPACE

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ maven_install(
5555
"io.netty:netty-transport-native-epoll:4.1.130.Final",
5656
"io.netty:netty-resolver:4.1.130.Final",
5757
"io.netty:netty-common:4.1.130.Final",
58+
"io.netty:netty-tcnative-boringssl-static:2.0.53.Final",
5859
"org.assertj:assertj-core:3.22.0",
5960
"org.mockito:mockito-core:3.10.0",
6061
"org.powermock:powermock-module-junit4:2.0.9",
@@ -128,6 +129,9 @@ maven_install(
128129
"com.alibaba.fastjson2:fastjson2:2.0.59",
129130
"org.junit.jupiter:junit-jupiter-api:5.9.1",
130131
],
132+
excluded_artifacts = [
133+
"com.google.protobuf:protobuf-javalite",
134+
],
131135
override_targets = {
132136
"com.google.protobuf:protobuf-javalite": "@maven//:com_google_protobuf_protobuf_java",
133137
},

proxy/BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -131,6 +131,7 @@ java_library(
131131
"@maven//:org_jetbrains_annotations",
132132
"@maven//:org_slf4j_jul_to_slf4j",
133133
"@maven//:commons_codec_commons_codec",
134+
"@maven//:io_netty_netty_tcnative_boringssl_static",
134135
],
135136
)
136137

0 commit comments

Comments
 (0)