Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Listen on Unix socket only; remove TCP listener and bring TypeScript to parity

Aperta
#35 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
45/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
docker, go, rust, typescript

Direzione di ricerca

The issue involves three codebases: Go, Rust, and TypeScript. Start by examining the existing listener code in each (e.g., ts/src/index.ts:43 for TypeScript, and similar locations for Go and Rust). Understand how Unix socket listening is implemented in Go and Rust to add it to TypeScript. Review the integration suite in deploy/ to update from TCP to Unix socket. Check flag handling (--listen-tcp, --listen-socket) and systemd socket activation. Done means all three implementations listen only on Unix sockets, the TCP flag is removed, and tests pass with the updated configuration.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Status: Break Change Type: Enhancement

Is your feature request related to a problem?

The proxy currently accepts requests over TCP as well as a Unix socket, which breaks its own security model.

Access control in this project is built on socket ownership: a caller is authorised because it can connect(2) to a Unix socket whose filesystem permissions (user/group) the operator controls. A TCP listener has no equivalent — there is no peer uid/gid, so anything that can reach the port is implicitly trusted. --listen-tcp defaults to 127.0.0.1:2375, and deploy/docker-compose.yml binds 0.0.0.0:2375, so the weaker path is the one exercised by default.

This is already acknowledged as a one-sided rule: all three implementations reject tcp:// for the outbound connection to the Docker daemon (ts/src/flags.ts:42, Makefile:111) on exactly these grounds. The inbound listener never got the same treatment.

The three implementations are also not equivalent today, which contradicts the "equal peers" rule in AGENTS.md:

Unix listener TCP listener
Go yes yes
Rust yes yes
TypeScript no yes

TypeScript is TCP-only (ts/src/index.ts:43) and has no Unix socket support at all.

Describe the solution

Make Unix socket the only transport the proxy listens on, in all three implementations.

  • Remove the --listen-tcp flag and its listener from Go, Rust, and TypeScript.
  • Add Unix socket listening to TypeScript via --listen-socket, reaching parity with Go and Rust, including fd://3 systemd socket activation and stale socket file cleanup.
  • Make a listen-socket bind failure fatal. With a single listener, a running-but-unbound process is never useful; today the bind error is logged and the process stays up.
  • Migrate the integration suite (deploy/docker-compose.yml, deploy/docker-compose.sock.yml, deploy/test.sh, deploy/test-sock.sh) from tcp://proxy:2375 to a shared-volume Unix socket.
  • Update README.md (flag table and security boundary section) and the systemd example, which currently documents ListenStream=127.0.0.1:2375.

Describe alternatives

  • Keep --listen-tcp but default it to disabled. Rejected: the insecure path stays reachable via config, and the security boundary stays advisory rather than structural.
  • Keep TCP and add peer authentication (mTLS). Rejected: substantially more surface and key management, to re-derive an identity the kernel already gives us for free over a Unix socket.
  • Leave TypeScript TCP-only. Rejected: directly contradicts the equal-peers rule.

Which implementation(s) would this affect?

  • Go
  • Rust
  • TypeScript
  • Quint specification
  • All

Additional context

Breaking change. Any deployment passing --listen-tcp or connecting via DOCKER_HOST=tcp://... must switch to a mounted Unix socket. The flag is removed outright rather than deprecated, so Go and Rust will exit on the unknown argument instead of silently listening on a socket the caller is not expecting.

The Quint spec models policy and invariants, not the transport layer, so no spec change is expected.

Lingua principale
Rust
Stelle
2
Fork
0
Merge medio
9h 35m
PR unite (30g)
6

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di ChainSafe/docker-socket-policy

Tutte le issue di ChainSafe/docker-socket-policy

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.