Skip to content

Commit a9a4fa9

Browse files
authored
Explain why ca-certificates is required in the Dockerfile (#50)
* Explain why ca-certificates is required in the Dockerfile The previous comment ("enables TLS/SSL for securely fetching dependencies") implied this is generic Docker hygiene. That framing is misleading: @livekit/rtc-node ships a native Rust core that reads the system trust store via rustls-tls-native-roots, not Node's bundled CA roots. node:22-slim doesn't ship /etc/ssl/certs/ca-certificates.crt, so without ca-certificates Room.connect() fails with the misleading "failed to retrieve region info" error. Rewrite the comment to name the cause so a developer trimming the image knows the line is load-bearing. No functional change. * Tighten ca-certificates comment * Restore TLS context and --no-install-recommends note; drop version-specific image name
1 parent e5c32a8 commit a9a4fa9

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

Dockerfile

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,10 @@ ENV PNPM_HOME="/pnpm"
1010
ENV PATH="$PNPM_HOME:$PATH"
1111
ENV HOME="/app"
1212

13-
# Install required system packages and pnpm, then clean up the apt cache for a smaller image
14-
# ca-certificates: enables TLS/SSL for securely fetching dependencies and calling HTTPS services
15-
# --no-install-recommends keeps the image minimal
13+
# Install ca-certificates (the system CA bundle used for TLS), then clean
14+
# the apt cache. Required by the LiveKit SDK: the native Rust core reads
15+
# the system trust store at runtime, which the slim base image doesn't ship.
16+
# --no-install-recommends keeps the image minimal.
1617
RUN apt-get update -qq && apt-get install --no-install-recommends -y ca-certificates && rm -rf /var/lib/apt/lists/*
1718

1819
# Pin pnpm version for reproducible builds

0 commit comments

Comments
 (0)