[incubator-pinot] branch master updated: [TE] frontend harleyjj/rca - double encode filter value to protect against special characters (#6281)

[jihao]

This is an automated email from the ASF dual-hosted git repository.

jihao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 5b0d1df  [TE] frontend harleyjj/rca - double encode filter value to 
protect against special characters (#6281)
5b0d1df is described below

commit 5b0d1df2563b2ad77d5ea0e8c2418c889769a989
Author: Harley Jackson <hjack...@linkedin.com>
AuthorDate: Mon Nov 30 22:38:55 2020 -0800

    [TE] frontend harleyjj/rca - double encode filter value to protect against 
special characters (#6281)
---
 thirdeye/thirdeye-frontend/app/utils/rca-utils.js | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/thirdeye/thirdeye-frontend/app/utils/rca-utils.js 
b/thirdeye/thirdeye-frontend/app/utils/rca-utils.js
index 622d144..1a491c5 100644
--- a/thirdeye/thirdeye-frontend/app/utils/rca-utils.js
+++ b/thirdeye/thirdeye-frontend/app/utils/rca-utils.js
@@ -184,11 +184,15 @@ export function appendTail(urn, tail) {
  * @see appendTail(urn, tail)
  *
  * @param {string} urn entity urn
- * @param {Array} filters array of filter tuples [key, value]
+ * @param {Array} filters array of strings ["key", "=", "value"]
  * @returns {string} merged metric urn
  */
 export function appendFilters(urn, filters) {
-  const tail = filters.map(t => encodeURIComponent(`${t[0]}${t[1]}${t[2]}`));
+  const tail = filters.map(t => {
+    // protects against special characters being in value string
+    const encodedValue = encodeURIComponent(t[2]);
+    return encodeURIComponent(`${t[0]}${t[1]}${encodedValue}`);
+  });
   return appendTail(urn, tail);
 }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org