exclude docker desktop

Author: Jacob Halsey

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "wsl2-dns-agent"
-version = "0.1.1"
+version = "0.2.0"
 edition = "2021"
 license = "GPL-3.0"
 description = "An agent that automatically patches your WSL2 DNS configuration for users of Cisco AnyConnect (or similar VPNs)"

Cargo.toml

@@ -46,3 +46,5 @@ shutdown = false
 [distributions.Ubuntu]
 apply_dns = false
 ```
+
+Note: the default configuration will ignore Docker Desktop, since the changes are unnecessary.

README.md

@@ -4,6 +4,8 @@ use std::collections::HashMap;
 use std::fs;
 use std::path::PathBuf;
 
+const EXCLUDE_BY_DEFAULT: &[&str] = &["docker-desktop", "docker-desktop-data"];
+
 #[derive(Debug, Deserialize, Serialize)]
 pub struct Config {
     /// Show toast notifications when DNS update is applied
@@ -13,10 +15,25 @@ pub struct Config {
     #[serde(default)]
     defaults: DistributionSetting,
     /// Per distribution settings
-    #[serde(default)]
+    #[serde(default = "default_distributions")]
     distributions: HashMap<String, DistributionSetting>,
 }
 
+fn default_distributions() -> HashMap<String, DistributionSetting> {
+    let mut map = HashMap::new();
+    for d in EXCLUDE_BY_DEFAULT {
+        map.insert(
+            d.to_string(),
+            DistributionSetting {
+                apply_dns: false,
+                shutdown: false,
+                patch_wsl_conf: false,
+            },
+        );
+    }
+    map
+}
+
 #[derive(Debug, Deserialize, Serialize)]
 pub struct DistributionSetting {
     /// Whether to update the wsl.conf and resolv.conf files for this distribution

src/config.rs

@@ -82,6 +82,8 @@ fn update_dns(config: &Config) -> Result<(), Error> {
             if let Err(e) = update_distribution(&d, dist_config, &resolv) {
                 log::error!("Failed to update DNS for {}, due to: {}", d.name, e);
             }
+        } else {
+            log::info!("Ignoring: {}", d.name);
         }
     }
     Ok(())