DevFW-CICD/ingress-nginx-helm
16
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Allow proxy-ssl-* annotations without proxy-ssl-secret

Browse source 
Signed-off-by: Tamal Saha <tamal@appscode.com>
This commit is contained in:
Tamal Saha 2023-10-23 17:32:47 -07:00
parent 0c3d52bade
commit 555c5eb04d
1 changed files with 18 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
internal/ingress/annotations/proxyssl/main.go
View file

@ -17,6 +17,7 @@ limitations under the License.
package proxyssl
import (
"errors"
"fmt"
"regexp"
"sort"
@ -190,10 +191,9 @@ func (p proxySSL) Parse(ing *networking.Ingress) (interface{}, error) {
config := &Config{}
proxysslsecret, err := parser.GetStringAnnotation(proxySSLSecretAnnotation, ing, p.annotationConfig.Annotations)
if err != nil {
if err != nil && !errors.Is(err, ing_errors.ErrMissingAnnotations) {
return &Config{}, err
}
} else if err == nil {
ns, _, err := k8s.ParseNameNS(proxysslsecret)
if err != nil {
return &Config{}, ing_errors.NewLocationDenied(err.Error())
@ -211,6 +211,7 @@ func (p proxySSL) Parse(ing *networking.Ingress) (interface{}, error) {
return &Config{}, ing_errors.LocationDeniedError{Reason: e}
}
config.AuthSSLCert = *proxyCert
}
config.Ciphers, err = parser.GetStringAnnotation(proxySSLCiphersAnnotation, ing, p.annotationConfig.Annotations)
if err != nil {