tlsutils: more test cases for OutgoingTLSConfigForCheck

This commit is contained in:
Daniel Nephin 2021-06-24 13:43:15 -04:00
parent 486b97e2c9
commit d09027caf6

View File

@ -937,6 +937,13 @@ func TestConfigurator_OutgoingTLSConfigForCheck(t *testing.T) {
} }
testCases := []testCase{ testCases := []testCase{
{
name: "default tls",
conf: func() (*Configurator, error) {
return NewConfigurator(Config{}, nil)
},
expected: &tls.Config{},
},
{ {
name: "default tls, skip verify, no server name", name: "default tls, skip verify, no server name",
conf: func() (*Configurator, error) { conf: func() (*Configurator, error) {
@ -955,6 +962,7 @@ func TestConfigurator_OutgoingTLSConfigForCheck(t *testing.T) {
TLSMinVersion: "tls12", TLSMinVersion: "tls12",
EnableAgentTLSForChecks: false, EnableAgentTLSForChecks: false,
ServerName: "servername", ServerName: "servername",
NodeName: "nodename",
}, nil) }, nil)
}, },
skipVerify: true, skipVerify: true,
@ -977,19 +985,34 @@ func TestConfigurator_OutgoingTLSConfigForCheck(t *testing.T) {
}, },
}, },
{ {
name: "agent tls, skip verify, default server name", name: "agent tls, default server name",
conf: func() (*Configurator, error) { conf: func() (*Configurator, error) {
return NewConfigurator(Config{ return NewConfigurator(Config{
TLSMinVersion: "tls12", TLSMinVersion: "tls12",
EnableAgentTLSForChecks: true, EnableAgentTLSForChecks: true,
NodeName: "nodename",
ServerName: "servername", ServerName: "servername",
}, nil) }, nil)
}, },
expected: &tls.Config{
MinVersion: tls.VersionTLS12,
ServerName: "servername",
},
},
{
name: "agent tls, skip verify, node name for server name",
conf: func() (*Configurator, error) {
return NewConfigurator(Config{
TLSMinVersion: "tls12",
EnableAgentTLSForChecks: true,
NodeName: "nodename",
}, nil)
},
skipVerify: true, skipVerify: true,
expected: &tls.Config{ expected: &tls.Config{
InsecureSkipVerify: true, InsecureSkipVerify: true,
MinVersion: tls.VersionTLS12, MinVersion: tls.VersionTLS12,
ServerName: "servername", ServerName: "nodename",
}, },
}, },
{ {