Skip to content
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

Jump to bottom

src,deps,build,test: add OpenSSL config appname #43124

Closed
wants to merge 2 commits into from
Closed

src,deps,build,test: add OpenSSL config appname #43124

wants to merge 2 commits into from

Commits on May 31, 2022

  1. src,deps,build,test: add OpenSSL config appname 

    This commit adds the setting of an appname (configuration section
name), 'nodejs_conf', to be used when reading OpenSSL configuration
files.

The motivation for this is that currently the default OpenSSL
configuration, 'openssl_conf', element will be used which may be
undesirable as it might configure OpenSSL in unwanted ways. With this
commit it is still possible to use a default openssl.cnf file but the
only section that Node.js will read from is a section named
'nodejs_conf'.

Refs: nodejs#40366
    @danbev
    danbev committed May 31, 2022
    Configuration menu
    Copy the full SHA
    04b9a1a View commit details
    Browse the repository at this point in the history

  2. src,doc,test: add --openssl-shared-config option 

    This commit adds a new command line option named
'--openssl-shared-config' intended to allow reverting to the old OpenSSL
configuration behavior where Node.js would use the configuration section
name (called appname in OpenSSL) 'openssl_conf' which could potentially
be used my other applications..
    @danbev
    danbev committed May 31, 2022
    Configuration menu
    Copy the full SHA
    495e72f View commit details
    Browse the repository at this point in the history