Skip to content

Commit

Permalink
Merge pull request #2143 from mazzi/fix_nat_only_in_auto
Browse files Browse the repository at this point in the history
fix(launchpad): nat mode only when first time automatic
  • Loading branch information
mazzi authored Sep 26, 2024
2 parents cd1b0a0 + dea6230 commit dd640b6
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion node-launchpad/src/components/status.rs
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,8 @@ impl Status {

/// Only run NAT detection if we haven't determined the status yet and we haven't failed more than 3 times.
fn should_we_run_nat_detection(&self) -> bool {
!self.is_nat_status_determined
self.connection_mode == ConnectionMode::Automatic
&& !self.is_nat_status_determined
&& self.error_while_running_nat_detection < MAX_ERRORS_WHILE_RUNNING_NAT_DETECTION
}

Expand Down

0 comments on commit dd640b6

Please sign in to comment.