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.

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

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

Closed
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 committed May 31, 2022
    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 committed May 31, 2022
    Copy the full SHA
    495e72f View commit details
    Browse the repository at this point in the history