[Backport 4.5.1-7.16] Fix agents active coverage stat as NaN (#5599)

Fix agents active coverage stat as NaN (#5490) * fix: agents active coverate stat as NaN Ensure the values used to calculate have the expected types and the total count is greater than 0. * remove: unused openRegistrationDocs method * changelog: add entry * fix: check if agents active coverage is a NaN * changelog: fix entry --------- Co-authored-by: Álex Ruiz <alejandro.ruiz.becerra@wazuh.com> (cherry picked from commit 53bf022) Co-authored-by: Ian Yenien Serrano <63758389+yenienserrano@users.noreply.github.com>
wazuh · Jun 28, 2023 · 19349eb · 19349eb
1 parent bb3d29b
commit 19349eb
Show file tree

Hide file tree

Showing 3 changed files with 212 additions and 108 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,7 @@ All notable changes to the Wazuh app project will be documented in this file.
 ### Fixed
 
 - Fixed the rendering of tables that contains IPs and agent overview [#5471](https://github.com/wazuh/wazuh-kibana-app/pull/5471)
+- Fixed the agents active coverage stat as NaN in Details panel of Agents section [#5490](https://github.com/wazuh/wazuh-kibana-app/pull/5490)
 
 ### Removed
 
@@ -28,6 +29,7 @@ All notable changes to the Wazuh app project will be documented in this file.
 
 ## Wazuh v4.4.4 - Kibana 7.10.2, 7.16.x, 7.17.x - Revision 01
 
+
 ### Added
 
 - Support for Wazuh 4.4.4

diff --git a/public/controllers/agent/agents-preview.js b/public/controllers/agent/agents-preview.js
@@ -35,7 +35,15 @@ export class AgentsPreviewController {
    * @param {Object} errorHandler
    * @param {Object} csvReq
    */
-  constructor($scope, $location, $route, errorHandler, csvReq, commonData, $window) {
+  constructor(
+    $scope,
+    $location,
+    $route,
+    errorHandler,
+    csvReq,
+    commonData,
+    $window,
+  ) {
     this.$scope = $scope;
     this.genericReq = GenericRequest;
     this.$location = $location;
@@ -57,11 +65,15 @@ export class AgentsPreviewController {
     this.api = JSON.parse(AppState.getCurrentAPI()).id;
     const loc = this.$location.search();
     if ((loc || {}).agent && (loc || {}).agent !== '000') {
-      this.commonData.setTimefilter(getDataPlugin().timefilter.timefilter.getTime());
+      this.commonData.setTimefilter(
+        getDataPlugin().timefilter.timefilter.getTime(),
+      );
       return this.showAgent({ id: loc.agent });
     }
 
-    this.isClusterEnabled = AppState.getClusterInfo() && AppState.getClusterInfo().status === 'enabled';
+    this.isClusterEnabled =
+      AppState.getClusterInfo() &&
+      AppState.getClusterInfo().status === 'enabled';
     this.loading = true;
     this.osPlatforms = [];
     this.versions = [];
@@ -82,22 +94,22 @@ export class AgentsPreviewController {
       this.$location.search('tab', this.submenuNavItem);
     });
 
-    this.$scope.$on('wazuhFetched', (evt) => {
+    this.$scope.$on('wazuhFetched', evt => {
       evt.stopPropagation();
     });
     this.registerAgentsProps = {
       addNewAgent: flag => this.addNewAgent(flag),
       hasAgents: () => this.hasAgents,
       reload: () => this.$route.reload(),
       getWazuhVersion: () => this.getWazuhVersion(),
-      getCurrentApiAddress: () => this.getCurrentApiAddress()
+      getCurrentApiAddress: () => this.getCurrentApiAddress(),
     };
     this.hasAgents = true;
     this.init = false;
     const instance = new DataFactory(WzRequest.apiReq, '/agents', false, false);
     //Props
     this.tableAgentsProps = {
-      updateSummary: (summary) => {
+      updateSummary: summary => {
         this.summary = summary;
         if (this.summary.total === 0) {
           if (this.addingNewAgent === undefined) {
@@ -117,18 +129,25 @@ export class AgentsPreviewController {
         this.downloadCsv(filters);
         this.$scope.$applyAsync();
       },
-      showAgent: (agent) => {
+      showAgent: agent => {
         this.showAgent(agent);
         this.$scope.$applyAsync();
       },
       getMostActive: async () => {
         return await this.getMostActive();
       },
       clickAction: (item, openAction = false) => {
-        clickAction(item, openAction, instance, this.shareAgent, this.$location, this.$scope);
+        clickAction(
+          item,
+          openAction,
+          instance,
+          this.shareAgent,
+          this.$location,
+          this.$scope,
+        );
         this.$scope.$applyAsync();
       },
-      formatUIDate: (date) => formatUIDate(date),
+      formatUIDate: date => formatUIDate(date),
       summary: this.summary,
     };
     //Load
@@ -187,14 +206,18 @@ export class AgentsPreviewController {
         'GET',
         `/elastic/top/${this.firstUrlParam}/${this.secondUrlParam}/agent.name/${
           this.pattern
-        }?agentsList=${store.getState().appStateReducers.allowedAgents.toString()}`
+        }?agentsList=${store
+          .getState()
+          .appStateReducers.allowedAgents.toString()}`,
       );
       this.mostActiveAgent.name = data.data.data;
       const info = await this.genericReq.request(
         'GET',
         `/elastic/top/${this.firstUrlParam}/${this.secondUrlParam}/agent.id/${
           this.pattern
-        }?agentsList=${store.getState().appStateReducers.allowedAgents.toString()}`
+        }?agentsList=${store
+          .getState()
+          .appStateReducers.allowedAgents.toString()}`,
       );
       if (info.data.data === '' && this.mostActiveAgent.name !== '') {
         this.mostActiveAgent.id = '000';
@@ -227,9 +250,12 @@ export class AgentsPreviewController {
     try {
       this.errorInit = false;
       const clusterInfo = AppState.getClusterInfo();
-      this.firstUrlParam = clusterInfo.status === 'enabled' ? 'cluster' : 'manager';
+      this.firstUrlParam =
+        clusterInfo.status === 'enabled' ? 'cluster' : 'manager';
       this.secondUrlParam = clusterInfo[this.firstUrlParam];
-      this.pattern = (await getDataPlugin().indexPatterns.get(AppState.getCurrentPattern())).title;
+      this.pattern = (
+        await getDataPlugin().indexPatterns.get(AppState.getCurrentPattern())
+      ).title;
     } catch (error) {
       const options = {
         context: `${AgentsPreviewController.name}.load`,
@@ -252,20 +278,14 @@ export class AgentsPreviewController {
     this.addingNewAgent = flag;
   }
 
-  openRegistrationDocs() {
-    this.$window.open(webDocumentationLink(user-manual/registering/index.html),
-      '_blank'
-    );
-  }
-
   /**
    * Returns the current API address
    */
   async getCurrentApiAddress() {
     try {
       const result = await this.genericReq.request('GET', '/hosts/apis');
       const entries = result.data || [];
-      const host = entries.filter((e) => {
+      const host = entries.filter(e => {
         return e.id == this.api;
       });
       const url = host[0].url;
@@ -279,7 +299,9 @@ export class AgentsPreviewController {
         error: {
           error: error,
           message: error.message || error,
-          title: `Could not get the Wazuh API address: ${error.message || error}`,
+          title: `Could not get the Wazuh API address: ${
+            error.message || error
+          }`,
         },
       };
       getErrorOrchestrator().handleError(options);