Skip to content

Commit

Permalink
doc: document that NODE_EXTRA_CA_CERTS is read only once
Browse files Browse the repository at this point in the history
Fixes: #20432

PR-URL: #37562
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
  • Loading branch information
jasnell authored and targos committed May 1, 2021
1 parent 77eb45a commit c21731b
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 1 deletion.
4 changes: 4 additions & 0 deletions doc/api/cli.md
Expand Up @@ -1175,6 +1175,10 @@ options property is explicitly specified for a TLS or HTTPS client or server.
This environment variable is ignored when `node` runs as setuid root or
has Linux file capabilities set.

The `NODE_EXTRA_CA_CERTS` environment variable is only read when the Node.js
process is first launched. Changing the value at runtime using
`process.env.NODE_EXTRA_CA_CERTS` has no effect on the current process.

### `NODE_ICU_DATA=file`
<!-- YAML
added: v0.11.15
Expand Down
7 changes: 7 additions & 0 deletions doc/node.1
Expand Up @@ -544,6 +544,13 @@ but any errors are otherwise ignored.
.Pp
This environment variable is ignored when `node` runs as setuid root or
has Linux file capabilities set.
.Pp
The
.Ar NODE_EXTRA_CA_CERTS
environment variable is only read when the Node.js process is first launched.
Changing the value at runtime using
.Ar process.env.NODE_EXTRA_CA_CERTS
has no effect on the current process.
.
.It Ev NODE_ICU_DATA Ar file
Data path for ICU (Intl object) data.
Expand Down
2 changes: 1 addition & 1 deletion lib/internal/main/print_help.js
Expand Up @@ -24,7 +24,7 @@ const envVars = new Map([
['NODE_DISABLE_COLORS', { helpText: 'set to 1 to disable colors in ' +
'the REPL' }],
['NODE_EXTRA_CA_CERTS', { helpText: 'path to additional CA certificates ' +
'file' }],
'file. Only read once during process startup.' }],
['NODE_NO_WARNINGS', { helpText: 'set to 1 to silence process warnings' }],
['NODE_PATH', { helpText: `'${require('path').delimiter}'-separated list ` +
'of directories prefixed to the module search path' }],
Expand Down

0 comments on commit c21731b

Please sign in to comment.