Skip to content
This repository has been archived by the owner on Aug 30, 2024. It is now read-only.

Commit

Permalink
fix: connected udp socket performance fix
Browse files Browse the repository at this point in the history
cloudflare#395

connected sock needs to be DGRAM

Signed-off-by: Tim Froidcoeur <[email protected]>
  • Loading branch information
tfroidcoeur committed Apr 3, 2024
1 parent 88a1473 commit 35fba29
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion boringtun/src/device/peer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ impl Peer {
.expect("Attempt to connect to undefined endpoint");

let udp_conn =
socket2::Socket::new(Domain::for_address(addr), Type::STREAM, Some(Protocol::UDP))?;
socket2::Socket::new(Domain::for_address(addr), Type::DGRAM, Some(Protocol::UDP))?;
udp_conn.set_reuse_address(true)?;
let bind_addr = if addr.is_ipv4() {
SocketAddrV4::new(Ipv4Addr::UNSPECIFIED, port).into()
Expand Down

0 comments on commit 35fba29

Please sign in to comment.