Squert: Priority counts incorrect #1277

.inc/callback.php

@@ -325,6 +325,10 @@ function level0() {
   $query->execute($merged_params);
   // fetch the data and encode to json
   $rows = $query->fetchAll(PDO::FETCH_ASSOC);
+  // the frontend expects all values to be strings
+  for ($i=0;$i<count($rows);$i++) {
+    $rows[$i] = array_map('strval', $rows[$i]);
+  }
   $theJSON = json_encode($rows);
   echo $theJSON;
 }
@@ -385,6 +389,10 @@ function level1() {
   $query->execute($merged_params);
   // fetch the data and encode to json
   $rows = $query->fetchAll(PDO::FETCH_ASSOC);
+  // the frontend expects all values to be strings
+  for ($i=0;$i<count($rows);$i++) {
+    $rows[$i] = array_map('strval', $rows[$i]);
+  }
   $theJSON = json_encode($rows);
   echo $theJSON;
 }

debian/changelog

@@ -1,3 +1,9 @@
+securityonion-squert (20161212-1ubuntu1securityonion42) xenial; urgency=medium
+
+  * Squert: Priority counts incorrect #1277 
+
+ -- Doug Burks <[email protected]>  Thu, 05 Jul 2018 07:06:30 -0400
+
 securityonion-squert (20161212-1ubuntu1securityonion41) xenial; urgency=medium
 
   * Issue 1259: Squert: turning grouping off results in no alerts 

debian/patches/Squert:-Priority-counts-incorrect-#1277

@@ -0,0 +1,49 @@
+Description: <short summary of the patch>
+ TODO: Put a short summary on the line above and replace this paragraph
+ with a longer explanation of this change. Complete the meta-information
+ with other relevant fields (see below for details). To make it easier, the
+ information below has been extracted from the changelog. Adjust it or drop
+ it.
+ .
+ securityonion-squert (20161212-1ubuntu1securityonion42) xenial; urgency=medium
+ .
+   * Squert: Priority counts incorrect #1277
+Author: Doug Burks <[email protected]>
+
+---
+The information above should follow the Patch Tagging Guidelines, please
+checkout http://dep.debian.net/deps/dep3/ to learn about the format. Here
+are templates for supplementary fields that you might want to add:
+
+Origin: <vendor|upstream|other>, <url of original patch>
+Bug: <url in upstream bugtracker>
+Bug-Debian: https://bugs.debian.org/<bugnumber>
+Bug-Ubuntu: https://launchpad.net/bugs/<bugnumber>
+Forwarded: <no|not-needed|url proving that it has been forwarded>
+Reviewed-By: <name and email of someone who approved the patch>
+Last-Update: <YYYY-MM-DD>
+
+--- securityonion-squert-20161212.orig/.inc/callback.php
++++ securityonion-squert-20161212/.inc/callback.php
+@@ -325,6 +325,10 @@ function level0() {
+   $query->execute($merged_params);
+   // fetch the data and encode to json
+   $rows = $query->fetchAll(PDO::FETCH_ASSOC);
++  // the frontend expects all values to be strings
++  for ($i=0;$i<count($rows);$i++) {
++    $rows[$i] = array_map('strval', $rows[$i]);
++  }
+   $theJSON = json_encode($rows);
+   echo $theJSON;
+ }
+@@ -385,6 +389,10 @@ function level1() {
+   $query->execute($merged_params);
+   // fetch the data and encode to json
+   $rows = $query->fetchAll(PDO::FETCH_ASSOC);
++  // the frontend expects all values to be strings
++  for ($i=0;$i<count($rows);$i++) {
++    $rows[$i] = array_map('strval', $rows[$i]);
++  }
+   $theJSON = json_encode($rows);
+   echo $theJSON;
+ }

debian/patches/series

@@ -36,3 +36,4 @@ update-mysql-function-calls-in-ip2c.php
 remove-unnecessary-code-from-ip2c.php:wq
 Issue-1169:-Squert:-remove-search-link-from-context-menu
 Issue-1259:-Squert:-turning-grouping-off-results-in-no-alerts
+Squert:-Priority-counts-incorrect-#1277