Skip to content

Commit 9106683

Browse files
committed
Enable SSL/TLS when allowed
Previously it was only enabled when mandatory.
1 parent 9a422b3 commit 9106683

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/facter/is_master.rb

+2-2
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,14 @@ def get_options_from_hash_config(config)
1616
# - sslMode is "requireSSL"
1717
# - Parameter --sslPEMKeyFile is set
1818
# - Parameter --sslCAFile is set
19-
result << "--ssl --host #{Facter.value(:fqdn)}" if config['net.ssl.mode'] == 'requireSSL' || !config['net.ssl.PEMKeyFile'].nil? || !config['net.ssl.CAFile'].nil?
19+
result << "--ssl --host #{Facter.value(:fqdn)}" if %w[allowSSL preferSSL requireSSL].include?(config['net.ssl.mode']) || !config['net.ssl.PEMKeyFile'].nil? || !config['net.ssl.CAFile'].nil?
2020
result << "--sslPEMKeyFile #{config['net.ssl.PEMKeyFile']}" unless config['net.ssl.PEMKeyFile'].nil?
2121
result << "--sslCAFile #{config['net.ssl.CAFile']}" unless config['net.ssl.CAFile'].nil?
2222
# use --tls and --host if:
2323
# - tlsMode is "requireTLS"
2424
# - Parameter --tlsCertificateKeyFile is set
2525
# - Parameter --tlsCAFile is set
26-
result << "--tls --host #{Facter.value(:fqdn)}" if config['net.tls.mode'] == 'requireTLS' || !config['net.tls.certificateKeyFile'].nil? || !config['net.tls.CAFile'].nil?
26+
result << "--tls --host #{Facter.value(:fqdn)}" if %w[allowTLS preferTLS requireTLS].include?(config['net.tls.mode']) || !config['net.tls.certificateKeyFile'].nil? || !config['net.tls.CAFile'].nil?
2727
result << "--tlsCertificateKeyFile #{config['net.tls.certificateKeyFile']}" unless config['net.tls.certificateKeyFile'].nil?
2828
result << "--tlsCAFile #{config['net.tls.CAFile']}" unless config['net.tls.CAFile'].nil?
2929

0 commit comments

Comments
 (0)