IntenseWebs/grafana
3
0
Fork 0
mirror of https://github.com/grafana/grafana.git synced 2025-02-25 18:55:37 -06:00
Code Issues Packages Projects Releases Wiki Activity

Tempo: Escape backslash in span name for promsql query (#83024)

Browse Source 
* Tempo: Escape backslash in span name for promsql query

In promsql, backslashes in regexes must be doubly escaped for them to be
interpreted as a literal backslash character.

Fixes #79791

* Fix regex replace

---------

Co-authored-by: Andrej Ocenas <mr.ocenas@gmail.com>
This commit is contained in:
ttshivers
2024-05-14 09:57:35 -05:00
committed by GitHub
parent 5f147644d4
commit b07b927a94
2 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
public/app/plugins/datasource/tempo/datasource.test.ts
View File

@@ -724,12 +724,14 @@ describe('Tempo service graph view', () => {
'/actuator/health/**',
'$type + [test]|HTTP POST - post',
'server.cluster.local:9090^/sample.test(.*)?',
'test\\path',
];
let escaped = getEscapedSpanNames(spanNames);
expect(escaped).toEqual([
'/actuator/health/\\\\*\\\\*',
'\\\\$type \\\\+ \\\\[test\\\\]\\\\|HTTP POST - post',
'server\\\\.cluster\\\\.local:9090\\\\^/sample\\\\.test\\\\(\\\\.\\\\*\\\\)\\\\?',
'test\\\\\\\\path',
]);
});

4
public/app/plugins/datasource/tempo/datasource.ts
View File

@@ -955,8 +955,10 @@ function makePromLink(title: string, expr: string, datasourceUid: string, instan
};
}
// TODO: this is basically the same as prometheus/datasource.ts#prometheusSpecialRegexEscape which is used to escape
// template variable values. It would be best to move it to some common place.
export function getEscapedSpanNames(values: string[]) {
return values.map((value: string) => value.replace(/[.*+?^${}()|[\]\\]/g, '\\\\$&'));
return values.map((value: string) => value.replace(/\\/g, '\\\\\\\\').replace(/[$^*{}\[\]\'+?.()|]/g, '\\\\$&'));
}
export function getFieldConfig(