CI fixes

Author: ammario

.github/workflows/tests.yml

@@ -79,11 +79,6 @@ jobs:
           chmod +x scripts/debug_tls_env.sh
           ./scripts/debug_tls_env.sh
           sudo ./scripts/debug_tls_env.sh
-          echo ""
-          echo "=== Testing Certificate Generation ==="
-          chmod +x scripts/debug_cert_generation.sh
-          ./scripts/debug_cert_generation.sh || true
-          echo "=== End Debug ==="
 
       - name: Run Linux jail integration tests (with sudo)
         run: |

src/jail/linux/mod.rs

@@ -68,10 +68,12 @@ pub struct LinuxJail {
 
 impl LinuxJail {
     pub fn new(config: JailConfig) -> Result<Self> {
-        // Use jail_id from config instead of generating our own
+        // Use jail_id from config for naming
+        // Note: Linux network interface names are limited to 15 characters
         let namespace_name = format!("httpjail_{}", config.jail_id);
-        let veth_host = format!("veth_h_{}", config.jail_id);
-        let veth_ns = format!("veth_n_{}", config.jail_id);
+        // Shorten to fit within 15 char limit: "vh_" + jail_id (max 10 chars)
+        let veth_host = format!("vh_{}", config.jail_id);
+        let veth_ns = format!("vn_{}", config.jail_id);
 
         Ok(Self {
             config,

src/jail/managed.rs

@@ -219,21 +219,6 @@ impl<J: Jail> ManagedJail<J> {
 
         Ok(Self { jail, lifecycle })
     }
-
-    /// Get a reference to the inner jail
-    pub fn inner(&self) -> &J {
-        &self.jail
-    }
-
-    /// Get a mutable reference to the inner jail
-    pub fn inner_mut(&mut self) -> &mut J {
-        &mut self.jail
-    }
-
-    /// Consume the managed jail and return the inner jail
-    pub fn into_inner(self) -> J {
-        self.jail
-    }
 }
 
 impl<J: Jail> Jail for ManagedJail<J> {

src/jail/mod.rs

@@ -8,6 +8,7 @@ pub mod managed;
 static JAIL_ID_COUNTER: AtomicU32 = AtomicU32::new(0);
 
 /// Trait for platform-specific jail implementations
+#[allow(dead_code)]
 pub trait Jail: Send + Sync {
     /// Setup jail for a specific session
     fn setup(&mut self, proxy_port: u16) -> Result<()>;

tests/system_integration.rs

@@ -464,6 +464,8 @@ pub fn test_concurrent_jail_isolation<P: JailTestPlatform>() {
 
     // Start first httpjail instance - allows only ifconfig.me
     let child1 = std::process::Command::new(&httpjail_path)
+        .arg("-v")
+        .arg("-v") // Add verbose logging to fix timing issues
         .arg("-r")
         .arg("allow: ifconfig\\.me")
         .arg("-r")
@@ -482,6 +484,8 @@ pub fn test_concurrent_jail_isolation<P: JailTestPlatform>() {
 
     // Start second httpjail instance - allows only ifconfig.io
     let output2 = std::process::Command::new(&httpjail_path)
+        .arg("-v")
+        .arg("-v") // Add verbose logging to fix timing issues
         .arg("-r")
         .arg("allow: ifconfig\\.io")
         .arg("-r")