Add Ingress TLS Hosts slice length check to avoid runtime panic (#358)

* Added check for Ingress TLS Hosts length to avoid runtime panic
In accordance with the issue #345 a check has been added to ensure that the Hosts slice in Ingress TLS is not accessed while it is empty.

* removed getTLSHosts() method

---------

Co-authored-by: Muhammad Shahid Hussain <34099012+LilShah@users.noreply.github.com>

pkg/kube/wrappers/ingress.go

@@ -99,7 +99,7 @@ func (iw *IngressWrapper) rulesExist() bool {
 }
 
 func (iw *IngressWrapper) tryGetTLSHost() (string, bool) {
-	if iw.supportsTLS() {
+	if iw.supportsTLS() && len(iw.ingress.Spec.TLS[0].Hosts) > 0 {
 		return "https://" + iw.ingress.Spec.TLS[0].Hosts[0], true
 	}
 

pkg/kube/wrappers/ingress_test.go

@@ -274,6 +274,14 @@ func TestIngressWrapper_tryGetTLSHost(t *testing.T) {
 			want:  "https://google.com",
 			want1: true,
 		},
+		{
+			name: "IngressWithTLSAndNoHosts",
+			fields: fields{
+				ingress: testutil.CreateIngressWithHostAndEmptyTLSHost("someIngress", "google.com"),
+			},
+			want:  "",
+			want1: false,
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {

pkg/testutil/kube.go

@@ -107,6 +107,17 @@ func CreateIngressWithHostAndTLSHost(name string, host string, tlsurl string) *v
 	return ingress
 }
 
+func CreateIngressWithHostAndEmptyTLSHost(name string, host string) *v1.Ingress {
+	ingress := CreateIngressWithHost(name, host)
+	ingress.Spec.TLS = []v1.IngressTLS{
+		{
+			Hosts: []string{},
+		},
+	}
+
+	return ingress
+}
+
 func CreateForecastleApp(name string, url string, group string, icon string) *v1alpha1.ForecastleApp {
 	return &v1alpha1.ForecastleApp{
 		ObjectMeta: metav1.ObjectMeta{