Merge pull request #579 from Abirdcfly/fixrange

Poorunga · web-flow · commit 0a727729a482 · 2026-03-13T23:57:50.000+08:00
fix panic due to empty gw
diff --git a/pkg/gateway/server.go b/pkg/gateway/server.go
@@ -161,7 +161,7 @@ func (srv *Server) newProto(conn net.Conn) (Protocol, error) {
 		// find all virtual services that bound to the gateway
 		fn := func(key, value interface{}) bool {
 			if vs, ok := value.(*istioapi.VirtualService); ok {
-				if vs.Spec.Gateways[0] == srv.options.GwName &&
+				if len(vs.Spec.Gateways) > 0 && vs.Spec.Gateways[0] == srv.options.GwName &&
 					reflect.DeepEqual(vs.Spec.Hosts, srv.options.Hosts) {
 					vss = append(vss, vs)
 				}

Original file line number	Diff line number	Diff line change
`@@ -161,7 +161,7 @@ func (srv *Server) newProto(conn net.Conn) (Protocol, error) {`
`161`	`161`	`// find all virtual services that bound to the gateway`
`162`	`162`	`fn := func(key, value interface{}) bool {`
`163`	`163`	`if vs, ok := value.(*istioapi.VirtualService); ok {`
`164`		`- if vs.Spec.Gateways[0] == srv.options.GwName &&`
	`164`	`+ if len(vs.Spec.Gateways) > 0 && vs.Spec.Gateways[0] == srv.options.GwName &&`
`165`	`165`	`reflect.DeepEqual(vs.Spec.Hosts, srv.options.Hosts) {`
`166`	`166`	`vss = append(vss, vs)`
`167`	`167`	`}`