fix: empty endpoint group check

gravitee-io · Dec 10, 2024 · 802ebc1 · 802ebc1
1 parent 7c3e9dd
commit 802ebc1
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/...m-console-webui/src/management/api/proxy/health-check/api-proxy-health-check.component.ts b/...m-console-webui/src/management/api/proxy/health-check/api-proxy-health-check.component.ts
@@ -75,7 +75,12 @@ export class ApiProxyHealthCheckComponent implements OnInit, OnDestroy {
         onlyApiV2Filter(this.snackBarService),
         switchMap((api) => {
           const apiHealthCheck = ApiProxyHealthCheckFormComponent.HealthCheckFromFormGroup(this.healthCheckForm, false);
-          this.updateEndpointsHealthCheckConfig(api.proxy?.groups);
+          try {
+            this.updateEndpointsHealthCheckConfig(api.proxy?.groups);
+          } catch (error) {
+            this.snackBarService.error('Failed to update configuration. Please ensure endpoint groups are properly defined.');
+            return EMPTY;
+          }
 
           return this.apiService.update(api.id, {
             ...api,
@@ -102,7 +107,7 @@ export class ApiProxyHealthCheckComponent implements OnInit, OnDestroy {
 
   updateEndpointsHealthCheckConfig(groups: Proxy['groups']) {
     groups.forEach((group) => {
-      group.endpoints.forEach((endpoint) => {
+      group.endpoints?.forEach((endpoint) => {
         // If healthcheck is disabled, set inherit to false
         if (
           (endpoint.healthCheck?.inherit === undefined || endpoint.healthCheck?.inherit === true) &&