Skip to content

Commit 1a0d955

Browse files
authored
Include also system CA certificates for Node in Docker image (#1565)
1 parent b3e80ec commit 1a0d955

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

.changeset/lucky-colts-laugh.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
'@graphql-hive/gateway': patch
3+
---
4+
5+
Include also system CA certificates for Node in Docker image
6+
7+
In light of the cert removals from https://nodejs.org/en/blog/release/v24.7.0

packages/gateway/node.Dockerfile

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,5 +49,8 @@ RUN echo "omit=peer" > .npmrc && chown node .npmrc
4949
# we need to set NODE_PATH to include because the root node_modules will dynamically import modules and we want node to search user-installed modules too (when extending the docker image)
5050
ENV NODE_PATH=/gateway/node_modules
5151

52+
# ensure that node uses the system CA certificates too because of https://nodejs.org/en/blog/release/v24.7.0
53+
ENV NODE_EXTRA_CA_CERTS=/etc/ssl/certs/ca-certificates.crt
54+
5255
USER node
5356
ENTRYPOINT ["dumb-init", "node", "bin.mjs"]

0 commit comments

Comments
 (0)