Skip to content

Commit 60785ea

Browse files
authored
Merge pull request #3 from Sentinel-Autonomybuilder/fix/windows-docker-pipe-detection
fix(docker): probe all Docker Desktop named pipes on Windows
2 parents 59f2cbb + 0e507e3 commit 60785ea

1 file changed

Lines changed: 10 additions & 1 deletion

File tree

src/main/services/docker.ts

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,16 @@ let client: Docker | null = null;
8787
function candidateSockets(override: string): string[] {
8888
if (override) return [override];
8989
if (process.platform === 'win32') {
90-
return ['//./pipe/docker_engine'];
90+
// Docker Desktop on Windows exposes different named pipes depending on
91+
// which context is active. `desktop-linux` (the default since 4.x) uses
92+
// `dockerDesktopLinuxEngine`; the legacy `default` context uses
93+
// `docker_engine`; Windows-containers mode uses `dockerDesktopWindowsEngine`.
94+
// Probe all three so a freshly-installed Docker Desktop works out of the box.
95+
return [
96+
'//./pipe/dockerDesktopLinuxEngine',
97+
'//./pipe/docker_engine',
98+
'//./pipe/dockerDesktopWindowsEngine',
99+
];
91100
}
92101
return [
93102
'/var/run/docker.sock',

0 commit comments

Comments
 (0)