Skip to content

Commit

Permalink
Remove 'force_wireguard_handshake' on non-Windows
Browse files Browse the repository at this point in the history
The firewall blocks the pinger in some circumstances
  • Loading branch information
dlon committed Dec 19, 2024
1 parent 977dd4a commit 3030cfe
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,8 @@ Line wrap the file at 100 chars. Th
### Changed
- (Linux and macOS only) Update to DAITA v2. The main difference is that many different machines are
provided by relays instead of a bundled list. The bundled `maybenot_machines` file was removed.

#### Windows
- Test tunnel before ephemeral peer exchange. This is an attempt to fix timeout issues.

### Fixed
Expand Down
2 changes: 1 addition & 1 deletion talpid-wireguard/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ fn main() {
// This is useful after updating the WireGuard config, to force a WireGuard handshake. This
// should reduce the number of PQ timeouts.
println!("cargo::rustc-check-cfg=cfg(force_wireguard_handshake)");
if matches!(target_os.as_str(), "linux" | "macos" | "windows") {
if target_os.as_str() == "windows" {
println!("cargo::rustc-cfg=force_wireguard_handshake");
}
}
Expand Down
4 changes: 3 additions & 1 deletion talpid-wireguard/src/connectivity/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ mod mock;
mod monitor;
mod pinger;

pub use check::{Cancellable, Check};
#[cfg(any(target_os = "android", force_wireguard_handshake))]
pub use check::Cancellable;
pub use check::Check;
pub use error::Error;
pub use monitor::Monitor;

0 comments on commit 3030cfe

Please sign in to comment.