From 8c35df5705d68d7706f40e7c7df06cd212943419 Mon Sep 17 00:00:00 2001 From: Frank Denis Date: Mon, 15 Mar 2021 16:18:27 +0100 Subject: [PATCH] Upgrade socket2 - Same thing, just breaking API changes --- Cargo.toml | 2 +- src/dnscrypt.rs | 7 +++---- src/main.rs | 37 +++++++++++++++++++------------------ 3 files changed, 23 insertions(+), 23 deletions(-) diff --git a/Cargo.toml b/Cargo.toml index de0b340..34e1e2a 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -27,7 +27,7 @@ ipext = "0.1.0" jemallocator = "0.3.2" libsodium-sys-stable= "1.19.14" log = { version = "0.4.14", features = ["std", "release_max_level_debug"] } -socket2 = "0.3.19" +socket2 = "0.4.0" parking_lot = "0.11.1" privdrop = "0.5.0" rand = "0.8.3" diff --git a/src/dnscrypt.rs b/src/dnscrypt.rs index 3272130..d35f04a 100644 --- a/src/dnscrypt.rs +++ b/src/dnscrypt.rs @@ -71,10 +71,9 @@ pub fn decrypt( let cached_shared_key = { let mut cache = dnscrypt_encryption_params.cache.as_ref().unwrap().lock(); - match cache.get(client_pk) { - None => None, - Some(cached_shared_key) => Some((*cached_shared_key).clone()), - } + cache + .get(client_pk) + .map(|cached_shared_key| (*cached_shared_key).clone()) }; let shared_key = match cached_shared_key { Some(cached_shared_key) => cached_shared_key, diff --git a/src/main.rs b/src/main.rs index 8835d57..7389ff2 100644 --- a/src/main.rs +++ b/src/main.rs @@ -3,6 +3,7 @@ #![allow(clippy::cognitive_complexity)] #![allow(clippy::upper_case_acronyms)] #![allow(clippy::unnecessary_wraps)] +#![allow(clippy::field_reassign_with_default)] #![allow(dead_code)] #[global_allocator] @@ -409,53 +410,53 @@ fn bind_listeners( ) -> Result, Error> { let mut sockets = Vec::with_capacity(listen_addrs.len()); for listen_addr in listen_addrs { - let tcp_listener = match listen_addr { + let tcp_listener: std::net::TcpListener = match listen_addr { SocketAddr::V4(_) => { let kindy = socket2::Socket::new( - socket2::Domain::ipv4(), - socket2::Type::stream(), - Some(socket2::Protocol::tcp()), + socket2::Domain::IPV4, + socket2::Type::STREAM, + Some(socket2::Protocol::TCP), )?; kindy.set_reuse_address(true)?; kindy.bind(&(*listen_addr).into())?; kindy.listen(TCP_BACKLOG as _)?; - kindy.into_tcp_listener() + kindy.into() } SocketAddr::V6(_) => { let kindy = socket2::Socket::new( - socket2::Domain::ipv6(), - socket2::Type::stream(), - Some(socket2::Protocol::tcp()), + socket2::Domain::IPV6, + socket2::Type::STREAM, + Some(socket2::Protocol::TCP), )?; kindy.set_reuse_address(true)?; kindy.set_only_v6(true)?; kindy.bind(&(*listen_addr).into())?; kindy.listen(TCP_BACKLOG as _)?; - kindy.into_tcp_listener() + kindy.into() } }; tcp_listener.set_nonblocking(true)?; - let udp_socket = match listen_addr { + let udp_socket: std::net::UdpSocket = match listen_addr { SocketAddr::V4(_) => { let kindy = socket2::Socket::new( - socket2::Domain::ipv4(), - socket2::Type::dgram(), - Some(socket2::Protocol::udp()), + socket2::Domain::IPV4, + socket2::Type::DGRAM, + Some(socket2::Protocol::UDP), )?; kindy.set_reuse_address(true)?; kindy.bind(&(*listen_addr).into())?; - kindy.into_udp_socket() + kindy.into() } SocketAddr::V6(_) => { let kindy = socket2::Socket::new( - socket2::Domain::ipv6(), - socket2::Type::dgram(), - Some(socket2::Protocol::udp()), + socket2::Domain::IPV6, + socket2::Type::DGRAM, + Some(socket2::Protocol::UDP), )?; kindy.set_reuse_address(true)?; kindy.set_only_v6(true)?; kindy.bind(&(*listen_addr).into())?; - kindy.into_udp_socket() + kindy.into() } }; udp_socket.set_nonblocking(true)?;