mirror of
https://github.com/pomerium/pomerium.git
synced 2025-06-20 11:38:07 +02:00
proxy: only set validation context if trusted_ca is used (#863)
* proxy: only set validation context if trusted_ca is used * fix test
This commit is contained in:
parent
9e711b4612
commit
fe2369400c
2 changed files with 14 additions and 12 deletions
|
@ -348,17 +348,22 @@ func buildDownstreamTLSContext(options *config.Options, domain string) *envoy_ex
|
|||
trustedCA = inlineFilename(options.ClientCAFile)
|
||||
}
|
||||
|
||||
var validationContext *envoy_extensions_transport_sockets_tls_v3.CommonTlsContext_ValidationContext
|
||||
if trustedCA != nil {
|
||||
validationContext = &envoy_extensions_transport_sockets_tls_v3.CommonTlsContext_ValidationContext{
|
||||
ValidationContext: &envoy_extensions_transport_sockets_tls_v3.CertificateValidationContext{
|
||||
TrustedCa: trustedCA,
|
||||
TrustChainVerification: envoy_extensions_transport_sockets_tls_v3.CertificateValidationContext_ACCEPT_UNTRUSTED,
|
||||
},
|
||||
}
|
||||
}
|
||||
|
||||
envoyCert := envoyTLSCertificateFromGoTLSCertificate(cert)
|
||||
return &envoy_extensions_transport_sockets_tls_v3.DownstreamTlsContext{
|
||||
CommonTlsContext: &envoy_extensions_transport_sockets_tls_v3.CommonTlsContext{
|
||||
TlsCertificates: []*envoy_extensions_transport_sockets_tls_v3.TlsCertificate{envoyCert},
|
||||
AlpnProtocols: []string{"h2", "http/1.1"},
|
||||
ValidationContextType: &envoy_extensions_transport_sockets_tls_v3.CommonTlsContext_ValidationContext{
|
||||
ValidationContext: &envoy_extensions_transport_sockets_tls_v3.CertificateValidationContext{
|
||||
TrustedCa: trustedCA,
|
||||
TrustChainVerification: envoy_extensions_transport_sockets_tls_v3.CertificateValidationContext_ACCEPT_UNTRUSTED,
|
||||
},
|
||||
},
|
||||
ValidationContextType: validationContext,
|
||||
},
|
||||
}
|
||||
}
|
||||
|
|
|
@ -313,10 +313,7 @@ func Test_buildDownstreamTLSContext(t *testing.T) {
|
|||
"filename": "`+keyFileName+`"
|
||||
}
|
||||
}
|
||||
],
|
||||
"validationContext": {
|
||||
"trustChainVerification": "ACCEPT_UNTRUSTED"
|
||||
}
|
||||
]
|
||||
}
|
||||
}`, downstreamTLSContext)
|
||||
}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue