diff --git a/app/server/appsmith-server/src/main/java/com/appsmith/server/solutions/ce/EnvManagerCEImpl.java b/app/server/appsmith-server/src/main/java/com/appsmith/server/solutions/ce/EnvManagerCEImpl.java index e09ff1709e..d5c8be34ae 100644 --- a/app/server/appsmith-server/src/main/java/com/appsmith/server/solutions/ce/EnvManagerCEImpl.java +++ b/app/server/appsmith-server/src/main/java/com/appsmith/server/solutions/ce/EnvManagerCEImpl.java @@ -458,16 +458,15 @@ public class EnvManagerCEImpl implements EnvManagerCE { // Generate analytics event properties template(s) according to the env variable changes List> analyticsEvents = getAnalyticsEvents(originalVariables, changes, new ArrayList<>()); + // Currently supporting only one authentication method update in one env update call + if (!analyticsEvents.isEmpty()) { + return analyticsService.sendObjectEvent(AnalyticsEvents.AUTHENTICATION_METHOD_CONFIGURATION, user, analyticsEvents.get(0)).then(); + } // We cannot send sensitive information present as values in env to the analytics // Values are filtered and only variable names are sent Map analyticsProperties = Map.of(FieldName.UPDATED_INSTANCE_SETTINGS, changes.keySet()); - Mono sendAnalyticsMono = Mono.empty(); - // Currently supporting only one authentication method update in one env update call - if (!analyticsEvents.isEmpty()) { - sendAnalyticsMono = analyticsService.sendObjectEvent(AnalyticsEvents.AUTHENTICATION_METHOD_CONFIGURATION, user, analyticsEvents.get(0)); - } - // A general INSTANCE_SETTING_UPDATED event is also sent for all admin settings changes - return sendAnalyticsMono.then(analyticsService.sendObjectEvent(AnalyticsEvents.INSTANCE_SETTING_UPDATED, user, analyticsProperties)).then(); + // A general INSTANCE_SETTING_UPDATED event is also sent for all admin settings changes other than Authentication method added/removed event + return analyticsService.sendObjectEvent(AnalyticsEvents.INSTANCE_SETTING_UPDATED, user, analyticsProperties).then(); } /**