Skip to content

fix: set NODE_EXTRA_CA_CERTS for Node.js gateway CA trust #25

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
johnnyfish wants to merge 1 commit into from
+6 −0
Closed

fix: set NODE_EXTRA_CA_CERTS for Node.js gateway CA trust #25

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
fix: set NODE_EXTRA_CA_CERTS for Node.js gateway CA trust
  • Loading branch information
@johnnyfish
johnnyfish committed May 7, 2026
commit 91ec7c09a8102499015fef1850a3589c37293566
6 changes: 6 additions & 0 deletions src/container/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,12 @@ export class ContainerClient {
`${hostCaPath}:${config.caCertificateContainerPath}:ro`,
);

// Node.js does not respect SSL_CERT_FILE; it merges NODE_EXTRA_CA_CERTS with its built-in CAs
args.push(
"-e",
`NODE_EXTRA_CA_CERTS=${config.caCertificateContainerPath}`,
);

// Build combined CA bundle for system-wide trust (curl, Python, Go, etc.)
if (combineCaBundle) {
const combinedPath = buildCombinedCaBundle(config.caCertificate);
Expand Down
Loading