Skip to content

Commit

Permalink
fix(docker_context): ignore unix domain socket path from Docker Conte…
Browse files Browse the repository at this point in the history
…xt (#5616)

* fix(modules): ignore unix domain socket path from Docker Context

fix #5548

* refactor(docker_context): ignore docker_context on unix domain socket path

Closes #5548
  • Loading branch information
ananta committed Dec 17, 2023
1 parent 00d3dc8 commit a910e09
Showing 1 changed file with 19 additions and 1 deletion.
20 changes: 19 additions & 1 deletion src/modules/docker_context.rs
Expand Up @@ -57,7 +57,7 @@ pub fn module<'a>(context: &'a Context) -> Option<Module<'a>> {
}
};

if ctx == "default" {
if ctx == "default" || ctx.starts_with("unix://") {
return None;
}

Expand Down Expand Up @@ -293,6 +293,24 @@ mod tests {
cfg_dir.close()
}

#[test]
fn test_docker_host_env_with_unix_path() -> io::Result<()> {
let cfg_dir = tempfile::tempdir()?;

let actual = ModuleRenderer::new("docker_context")
.env("DOCKER_HOST", "unix:///run/user/1001/podman/podman.sock")
.config(toml::toml! {
[docker_context]
only_with_files = false
})
.collect();
let expected = None;

assert_eq!(expected, actual);

cfg_dir.close()
}

#[test]
fn test_docker_context_env() -> io::Result<()> {
let cfg_dir = tempfile::tempdir()?;
Expand Down

0 comments on commit a910e09

Please sign in to comment.