vopono v0.6.1 - Hotfix for pactl parse errors

Author: jamesmcm

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "vopono"
 description = "Launch applications via VPN tunnels using temporary network namespaces"
-version = "0.6.0"
+version = "0.6.1"
 authors = ["James McMurray <[email protected]>"]
 edition = "2018"
 license = "GPL-3.0-or-later"

src/main.rs

@@ -22,7 +22,7 @@ mod wireguard;
 
 use list::output_list;
 use list_configs::print_configs;
-use log::{debug, LevelFilter};
+use log::{debug, warn, LevelFilter};
 use netns::NetworkNamespace;
 use structopt::StructOpt;
 use sync::{sync_menu, synch};
@@ -52,8 +52,15 @@ fn main() -> anyhow::Result<()> {
         args::Command::Exec(cmd) => {
             clean_dead_locks()?;
             if which("pactl").is_ok() {
-                let pa = pulseaudio::get_pulseaudio_server()?;
-                std::env::set_var("PULSE_SERVER", pa);
+                let pa = pulseaudio::get_pulseaudio_server();
+                if let Ok(pa) = pa {
+                    std::env::set_var("PULSE_SERVER", pa);
+                } else {
+                    warn!(
+                        "Could not parse PULSE_SERVER from pactl info output: {:?}",
+                        pa
+                    );
+                }
             } else {
                 debug!("pactl not found, will not set PULSE_SERVER");
             }

src/pulseaudio.rs

@@ -10,11 +10,11 @@ pub fn get_pulseaudio_server() -> anyhow::Result<String> {
 
     let caps = re.captures(output);
     if caps.is_none() {
-        return Err(anyhow!("Could not parse pactl output!"));
+        return Err(anyhow!("Could not parse pactl output!:\n{}", output));
     }
     let caps = caps.unwrap().get(1);
     if caps.is_none() {
-        return Err(anyhow!("Could not parse pactl output!"));
+        return Err(anyhow!("Could not parse pactl output!:\n{}", output));
     }
 
     let out = caps.unwrap().as_str().to_string();