Skip to content

Commit

Permalink
check sslpolicy on both sdkLS and resLS (kubernetes-sigs#3196)
Browse files Browse the repository at this point in the history
  • Loading branch information
oliviassss authored May 15, 2023
1 parent 5317c41 commit 1645277
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/deploy/elbv2/listener_manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -158,8 +158,8 @@ func (m *defaultListenerManager) updateSDKListenerWithSettings(ctx context.Conte
func (m *defaultListenerManager) updateSDKListenerWithExtraCertificates(ctx context.Context, resLS *elbv2model.Listener,
sdkLS ListenerWithTags, isNewSDKListener bool) error {
// if TLS is not supported, we shouldn't update
if sdkLS.Listener.SslPolicy == nil {
m.logger.V(1).Info("SDK Listener doesn't have SSL Policy set, we skip updating extra certs for non-TLS listener.")
if resLS.Spec.SSLPolicy == nil && sdkLS.Listener.SslPolicy == nil {
m.logger.V(1).Info("Res and Sdk Listener don't have SSL Policy set, we skip updating extra certs for non-TLS listener.")
return nil
}

Expand Down

0 comments on commit 1645277

Please sign in to comment.