Skip to content

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 ddd5b73 commit 0887d7c
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)?;
// SO_REUSEPORT is not really needed here, as long as the wildcard address is used in the bind
udp_conn.set_reuse_port(true)?;
Expand Down

0 comments on commit 0887d7c

Please sign in to comment.