From 7e9db1d8766c53d8c6dbd2ac45a26011febcfcb0 Mon Sep 17 00:00:00 2001 From: Reactor Scram Date: Fri, 5 Jul 2024 03:16:33 +0000 Subject: [PATCH] chore(headless-client): fix typo in match statement (#5706) PR #5700 had a typo in it. I didn't notice that these match arms use `|`, so I accidentally flush the DNS for an event that doesn't need it. Only `OnUpdateResources` should flush DNS. --- rust/headless-client/src/standalone.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/rust/headless-client/src/standalone.rs b/rust/headless-client/src/standalone.rs index 5d0a70575..991e54dc9 100644 --- a/rust/headless-client/src/standalone.rs +++ b/rust/headless-client/src/standalone.rs @@ -191,8 +191,8 @@ pub fn run_only_headless_client() -> Result<()> { is_authentication_error: _, }) => return Err(anyhow!(error_msg).context("Firezone disconnected")), InternalServerMsg::Ipc(IpcServerMsg::Ok) - | InternalServerMsg::Ipc(IpcServerMsg::OnTunnelReady) - | InternalServerMsg::Ipc(IpcServerMsg::OnUpdateResources(_)) => { + | InternalServerMsg::Ipc(IpcServerMsg::OnTunnelReady) => {} + InternalServerMsg::Ipc(IpcServerMsg::OnUpdateResources(_)) => { // On every resources update, flush DNS to mitigate dns_controller.flush()?; }