Merge pull request #659 from openziti/fix-service-list-handler

Move service list handler invoke out of decode loop

router/handler_edge_ctrl/serviceList.go

@@ -66,6 +66,6 @@ func (self *ServiceListHandler) handleServicesList(list *edge_ctrl_pb.ServicesLi
 		}
 
 		serviceList = append(serviceList, service)
-		self.handler(list.LastUpdate, serviceList)
 	}
+	self.handler(list.LastUpdate, serviceList)
 }

router/xgress_edge_tunnel/servicepoll.go

@@ -80,7 +80,7 @@ func (self *servicePoller) handleServiceListUpdate(lastUpdateToken []byte, servi
 
 	// Adds and Updates
 	for _, s := range services {
-		self.services.Upsert(s.Name, s, func(exist bool, valueInMap interface{}, newValue interface{}) interface{} {
+		self.services.Upsert(s.Id, s, func(exist bool, valueInMap interface{}, newValue interface{}) interface{} {
 			if !exist {
 				self.serviceListener.HandleServicesChange(ziti.ServiceAdded, s)
 				return s