Merge pull request #6861 from Countly/master-v25.03.25-into-next

Master v25.03.25 into next

Author: gabrieloliveirapinto

CHANGELOG.md

@@ -2,6 +2,15 @@
 Dependencies:
 - Remove SQLite
 
+## Version 25.03.25
+Fixes:
+- [crashes] Fixed resolving audit log recording
+- [location] Fixed updating none gps coordinate location after gps location was used
+- 
+Enterprise Fixes:
+- [ab-testing] Add script for fixing variant cohort
+- [groups] Fix user permission update after updating user group permission
+
 ## Version 25.03.24
 Fixes:
 - [jobs] Fix condition for scheduling alert job

api/parts/data/usage.js

@@ -1030,7 +1030,7 @@ plugins.register("/sdk/user_properties", async function(ob) {
 
                 if (plugins.getConfig('api', params.app && params.app.plugins, true).city_data === true && !userProps.loc && typeof data.lat !== "undefined" && typeof data.lon !== "undefined") {
                     // only override lat/lon if no recent gps location exists in user document
-                    if (!params.app_user.loc || (params.app_user.loc.gps && params.time.mstimestamp - params.app_user.loc.date > 7 * 24 * 3600)) {
+                    if (!params.app_user.loc || !params.app_user.loc.gps || params.time.mstimestamp - params.app_user.loc.date > 7 * 24 * 3600) {
                         userProps.loc = {
                             gps: false,
                             geo: {
@@ -1061,7 +1061,7 @@ plugins.register("/sdk/user_properties", async function(ob) {
 
                     if (plugins.getConfig('api', params.app && params.app.plugins, true).city_data === true && !userProps.loc && data.ll && typeof data.ll[0] !== "undefined" && typeof data.ll[1] !== "undefined") {
                         // only override lat/lon if no recent gps location exists in user document
-                        if (!params.app_user.loc || (params.app_user.loc.gps && params.time.mstimestamp - params.app_user.loc.date > 7 * 24 * 3600)) {
+                        if (!params.app_user.loc || !params.app_user.loc.gps || params.time.mstimestamp - params.app_user.loc.date > 7 * 24 * 3600) {
                             userProps.loc = {
                                 gps: false,
                                 geo: {

bin/scripts/fix-data/recalculate_ab_test_cohorts.js

@@ -0,0 +1,74 @@
+/*
+ *  Sends recalculate request for all ab testing experiment variant cohorts
+ *  Server: countly
+ *  Path: $(countly dir)/bin/scripts/fix-data
+ *  Command: node recalculate_ab_test_cohorts.js
+ */
+
+// API key here with permission to update cohorts
+const API_KEY = '';
+// Countly app id, if not specified will do nothing
+const APP_ID = '';
+// ab test experiment id, will do nothing if not specified
+const EXPERIMENT_ID = '';
+// countly instance public url, something like 'https://name.count.ly'
+const SERVER_URL = '';
+
+const pluginManager = require('../../../plugins/pluginManager.js');
+const request = require('countly-request')(pluginManager.getConfig('security'));
+
+if (API_KEY.length === 0) {
+    console.warn('Please provide an API_KEY');
+    process.exit(1);
+}
+
+pluginManager.dbConnection('countly_out').then(async(db) => {
+    let urlObj = {};
+    try {
+        urlObj = new URL(SERVER_URL);
+    }
+    catch (err) {
+        urlObj = new URL((process.env.COUNTLY_CONFIG_PROTOCOL || "http") + "://" + (process.env.COUNTLY_CONFIG_HOSTNAME || "localhost"));
+    }
+    urlObj.pathname = 'i/cohorts/recalculate';
+    urlObj.searchParams.append('api_key', API_KEY);
+    urlObj.searchParams.append('app_id', APP_ID);
+
+    console.log(`Finding ab test experiment ${EXPERIMENT_ID} in app ${APP_ID}`);
+
+    const experimentCollectionName = `ab_testing_experiments${APP_ID}`;
+    const experiment = await db.collection(experimentCollectionName).findOne({ _id: db.ObjectID(EXPERIMENT_ID) });
+
+    if (experiment?.variants?.length > 0) {
+        for (let varIdx = 0; varIdx < experiment.variants.length; varIdx += 1) {
+            const variant = experiment.variants[varIdx];
+
+            if (variant?.cohorts && Object.keys(variant.cohorts).length > 0) {
+                for (let cohIdx = 0; cohIdx < Object.keys(variant.cohorts).length; cohIdx += 1) {
+                    const cohortId = variant.cohorts[Object.keys(variant.cohorts)[cohIdx]];
+                    console.log(`Sending recalculate request for variant ${variant.name}, cohort ${cohortId}`);
+
+                    urlObj.searchParams.delete('cohort_id');
+                    urlObj.searchParams.append('cohort_id', cohortId);
+
+                    await new Promise((resolve) => {
+                        request.get(urlObj.href, (err, _, body) => {
+                            if (err) {
+                                console.warn('Request failed ', JSON.stringify(cohortId), err);
+                            }
+                            else {
+                                console.log('Request finished ', JSON.stringify(cohortId), body);
+                            }
+                            resolve();
+                        });
+                    });
+                }
+            }
+        }
+    }
+    else {
+        console.warn(`Experiments ${EXPERIMENT_ID} not found in app ${APP_ID}`);
+    }
+
+    db.close();
+});

frontend/express/public/core/user-management/javascripts/countly.views.js

@@ -866,6 +866,7 @@
                 }
             },
             onOpen: function() {
+                this.patchUserPermission();
                 this.changePasswordFlag = false;
                 // types
                 var types = ['c', 'r', 'u', 'd'];
@@ -879,7 +880,7 @@
                 // if it's in edit mode
                 if (this.settings.editMode) {
                     // is user member of a group?
-                    if (this.user.group_id && countlyGlobal.plugins.indexOf('groups') > -1) {
+                    if (this.user.group_id && this.user.group_id.length && countlyGlobal.plugins.indexOf('groups') > -1) {
                         // set groups state
                         if (Array.isArray(this.user.group_id)) {
                             this.groups = this.user.group_id;
@@ -982,7 +983,31 @@
             },
             onRoleChange: function(role) {
                 this.roles[role.name] = role;
-            }
+            },
+            patchUserPermission: function() {
+                if (this.user && this.user.permission && this.user.permission._ && this.user.permission._.u) {
+                    var appIdReducer = function(acc, curr) {
+                        if (curr.length > 0) {
+                            acc.push(curr);
+                        }
+                        return acc;
+                    };
+
+                    var _u = this.user.permission._.u.reduce(function(acc, curr) {
+                        if (curr.length > 0) {
+                            var appIds = curr.reduce(appIdReducer, []);
+
+                            if (appIds.length > 0) {
+                                acc.push(appIds);
+                            }
+                        }
+                        return acc;
+                    }, []);
+
+                    this.user.permission._.u = _u;
+                    this.$refs.userDrawer.editedObject.permission._.u = _u;
+                }
+            },
         },
         watch: {
             'groups': function() {