mirror of
https://github.com/nodejs/node.git
synced 2025-05-14 15:26:29 +00:00
build: disable SSLv2 by default
No one in this day and age should be using SSLv2 so disable it by default. You can re-enable it with `./configure --with-sslv2` but there really should be no reason for that.
This commit is contained in:
parent
c937f5b587
commit
39aa894035
14
configure
vendored
14
configure
vendored
@ -54,11 +54,6 @@ parser.add_option('--no-ifaddrs',
|
|||||||
dest='no_ifaddrs',
|
dest='no_ifaddrs',
|
||||||
help='use on deprecated SunOS systems that do not support ifaddrs.h')
|
help='use on deprecated SunOS systems that do not support ifaddrs.h')
|
||||||
|
|
||||||
parser.add_option('--no-ssl2',
|
|
||||||
action='store_true',
|
|
||||||
dest='no_ssl2',
|
|
||||||
help='disable OpenSSL v2')
|
|
||||||
|
|
||||||
# deprecated
|
# deprecated
|
||||||
parser.add_option('--openssl-includes',
|
parser.add_option('--openssl-includes',
|
||||||
action='store',
|
action='store',
|
||||||
@ -230,6 +225,11 @@ parser.add_option('--with-perfctr',
|
|||||||
dest='with_perfctr',
|
dest='with_perfctr',
|
||||||
help='build with performance counters (default is true on Windows)')
|
help='build with performance counters (default is true on Windows)')
|
||||||
|
|
||||||
|
parser.add_option('--with-sslv2',
|
||||||
|
action='store_true',
|
||||||
|
dest='with_sslv2',
|
||||||
|
help='enable SSL v2')
|
||||||
|
|
||||||
parser.add_option('--without-dtrace',
|
parser.add_option('--without-dtrace',
|
||||||
action='store_true',
|
action='store_true',
|
||||||
dest='without_dtrace',
|
dest='without_dtrace',
|
||||||
@ -588,7 +588,9 @@ def configure_openssl(o):
|
|||||||
if options.without_ssl:
|
if options.without_ssl:
|
||||||
return
|
return
|
||||||
|
|
||||||
if options.no_ssl2:
|
# OpenSSL uses `#ifndef OPENSSL_NO_SSL2` checks so only define the
|
||||||
|
# macro when we want to _disable_ SSL2.
|
||||||
|
if not options.with_sslv2:
|
||||||
o['defines'] += ['OPENSSL_NO_SSL2=1']
|
o['defines'] += ['OPENSSL_NO_SSL2=1']
|
||||||
|
|
||||||
if options.shared_openssl:
|
if options.shared_openssl:
|
||||||
|
Loading…
Reference in New Issue
Block a user