Skip to content

Commit 473c8d0

Browse files
fix build
1 parent f657387 commit 473c8d0

File tree

4 files changed

+33
-36
lines changed

4 files changed

+33
-36
lines changed

libp2p/src/main/kotlin/io/libp2p/transport/implementation/PlainNettyTransport.kt

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,8 @@ import io.netty.bootstrap.ServerBootstrap
1919
import io.netty.channel.Channel
2020
import io.netty.channel.ChannelHandler
2121
import io.netty.channel.ChannelOption
22-
import io.netty.channel.nio.NioEventLoopGroup
22+
import io.netty.channel.MultiThreadIoEventLoopGroup
23+
import io.netty.channel.nio.NioIoHandler
2324
import io.netty.channel.socket.nio.NioServerSocketChannel
2425
import io.netty.channel.socket.nio.NioSocketChannel
2526
import java.net.InetSocketAddress
@@ -39,8 +40,12 @@ abstract class PlainNettyTransport(
3940
private val listeners = mutableMapOf<Multiaddr, Channel>()
4041
private val channels = mutableListOf<Channel>()
4142

42-
private var workerGroup by lazyVar { NioEventLoopGroup() }
43-
private var bossGroup by lazyVar { NioEventLoopGroup(1) }
43+
private var workerGroup by lazyVar {
44+
MultiThreadIoEventLoopGroup(NioIoHandler.newFactory())
45+
}
46+
private var bossGroup by lazyVar {
47+
MultiThreadIoEventLoopGroup(1, NioIoHandler.newFactory())
48+
}
4449

4550
private var client by lazyVar {
4651
Bootstrap().apply {

libp2p/src/main/kotlin/io/libp2p/transport/quic/QuicTransport.kt

Lines changed: 21 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,11 @@ import io.netty.buffer.PooledByteBufAllocator
3232
import io.netty.channel.*
3333
import io.netty.channel.epoll.Epoll
3434
import io.netty.channel.epoll.EpollDatagramChannel
35-
import io.netty.channel.nio.NioEventLoopGroup
35+
import io.netty.channel.nio.NioIoHandler
3636
import io.netty.channel.socket.nio.NioDatagramChannel
3737
import io.netty.handler.codec.quic.*
3838
import io.netty.handler.ssl.ClientAuth
39+
import org.slf4j.LoggerFactory
3940
import java.net.InetSocketAddress
4041
import java.net.SocketAddress
4142
import java.time.Duration
@@ -49,14 +50,20 @@ class QuicTransport(
4950
private val protocols: List<ProtocolBinding<*>>
5051
) : NettyTransport {
5152

53+
private val logger = LoggerFactory.getLogger(QuicTransport::class.java)
54+
5255
private var closed = false
5356
var connectTimeout = Duration.ofSeconds(15)
5457

5558
private val listeners = mutableMapOf<Multiaddr, Channel>()
5659
private val channels = mutableListOf<Channel>()
5760

58-
private var workerGroup by lazyVar { NioEventLoopGroup() }
59-
private var bossGroup by lazyVar { NioEventLoopGroup(1) }
61+
private var workerGroup by lazyVar {
62+
MultiThreadIoEventLoopGroup(NioIoHandler.newFactory())
63+
}
64+
private var bossGroup by lazyVar {
65+
MultiThreadIoEventLoopGroup(1, NioIoHandler.newFactory())
66+
}
6067
private var allocator by lazyVar { PooledByteBufAllocator(true) }
6168
private var multistreamProtocol: MultistreamProtocol = MultistreamProtocolV1
6269
private var incomingMultistreamProtocol: MultistreamProtocol by lazyVar { multistreamProtocol }
@@ -166,7 +173,7 @@ class QuicTransport(
166173
listeners -= addr
167174
}
168175
}
169-
println("Quic server listening on " + addr)
176+
logger.info("Quic server listening on $addr")
170177
res.complete(null)
171178
}
172179
}
@@ -300,38 +307,23 @@ class QuicTransport(
300307

301308
override fun handles(addr: Multiaddr) =
302309
handlesHost(addr) &&
303-
addr.has(UDP) &&
304-
addr.has(QUICV1) &&
305-
!addr.has(WS)
310+
addr.has(UDP) &&
311+
addr.has(QUICV1) &&
312+
!addr.has(WS)
306313

307314
fun quicSslContext(expectedRemotePeerId: PeerId?, trustManager: Libp2pTrustManager): QuicSslContext {
308315
val connectionKeys = if (certAlgorithm.equals("ECDSA")) generateEcdsaKeyPair() else generateEd25519KeyPair()
309316
val javaPrivateKey = getJavaKey(connectionKeys.first)
310317
val isClient = expectedRemotePeerId != null
311318
val cert = buildCert(localKey, connectionKeys.first)
312-
println("Building " + certAlgorithm + " keys and cert for peerid " + PeerId.fromPubKey(localKey.publicKey()))
319+
logger.debug("Building {} keys and cert for peerid {}", certAlgorithm, PeerId.fromPubKey(localKey.publicKey()))
313320
return (
314-
if (isClient) {
315-
QuicSslContextBuilder.forClient().keyManager(javaPrivateKey, null, cert)
316-
} else {
317-
QuicSslContextBuilder.forServer(javaPrivateKey, null, cert).clientAuth(ClientAuth.REQUIRE)
318-
}
319-
)
320-
// .option(BoringSSLContextOption.GROUPS, arrayOf("x25519"))
321-
// .option(
322-
// BoringSSLContextOption.SIGNATURE_ALGORITHMS,
323-
// arrayOf(
324-
// // "ed25519",
325-
// "ecdsa_secp256r1_sha256",
326-
// "rsa_pkcs1_sha256",
327-
// "rsa_pss_rsae_sha256",
328-
// "ecdsa_secp384r1_sha384",
329-
// "rsa_pkcs1_sha384",
330-
// "rsa_pss_rsae_sha384",
331-
// "rsa_pss_rsae_sha512",
332-
// "rsa_pkcs1_sha512",
333-
// )
334-
// )
321+
if (isClient) {
322+
QuicSslContextBuilder.forClient().keyManager(javaPrivateKey, null, cert)
323+
} else {
324+
QuicSslContextBuilder.forServer(javaPrivateKey, null, cert).clientAuth(ClientAuth.REQUIRE)
325+
}
326+
)
335327
.trustManager(trustManager)
336328
.applicationProtocols("libp2p")
337329
.build()

libp2p/src/main/kotlin/io/libp2p/transport/ws/WebSocketClientInitializer.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,12 @@ internal class WebSocketClientInitializer(
1212
private val url: String
1313
) : ChannelInitializer<SocketChannel>() {
1414

15-
public override fun initChannel(ch: SocketChannel) {
15+
override fun initChannel(ch: SocketChannel) {
1616
val pipeline = ch.pipeline()
1717

1818
pipeline.addLast(HttpClientCodec())
1919
pipeline.addLast(HttpObjectAggregator(65536))
20-
pipeline.addLast(WebSocketClientCompressionHandler.INSTANCE)
20+
pipeline.addLast(WebSocketClientCompressionHandler(0))
2121
pipeline.addLast(
2222
WebSocketClientHandshake(
2323
connectionBuilder,

libp2p/src/main/kotlin/io/libp2p/transport/ws/WebSocketServerInitializer.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,12 @@ internal class WebSocketServerInitializer(
1212
private val connectionBuilder: ChannelHandler
1313
) : ChannelInitializer<SocketChannel>() {
1414

15-
public override fun initChannel(ch: SocketChannel) {
15+
override fun initChannel(ch: SocketChannel) {
1616
val pipeline = ch.pipeline()
1717

1818
pipeline.addLast(HttpServerCodec())
1919
pipeline.addLast(HttpObjectAggregator(65536))
20-
pipeline.addLast(WebSocketServerCompressionHandler())
20+
pipeline.addLast(WebSocketServerCompressionHandler(0))
2121
pipeline.addLast(WebSocketServerProtocolHandler("/", null, true))
2222
pipeline.addLast(WebSocketServerHandshakeListener(connectionBuilder))
2323
} // initChannel

0 commit comments

Comments
 (0)