Handle errors in regex match

komodorio · Jan 15, 2023 · 87ee388 · 87ee388
1 parent f3c66ec
commit 87ee388
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/pkg/dashboard/static/list-view.js b/pkg/dashboard/static/list-view.js
@@ -45,9 +45,15 @@ function buildChartCard(elm) {
         </div>`)
 
     let chartName = elm.chart
+    let match = null
     // semver2 regex , add optional v prefix
     const chartNameRegex = 'v?(0|[1-9]\\d*)\\.(0|[1-9]\\d*)\\.(0|[1-9]\\d*)(?:-((?:0|[1-9]\\d*|\\d*[a-zA-Z-][0-9a-zA-Z-]*)(?:\\.(?:0|[1-9]\\d*|\\d*[a-zA-Z-][0-9a-zA-Z-]*))*))?(?:\\+([0-9a-zA-Z-]+(?:\\.[0-9a-zA-Z-]+)*))?'
-    const match = elm.chart.match(chartNameRegex);
+    if (!new RegExp(chartNameRegex).test(chartName)) {
+        alert('Chart name does not match chart name regex.')
+    } else {
+        match = chartName.match(chartNameRegex);
+    }
+
     if (match) {
         chartName = elm.chart.substring(0, match.index - 1)
     } else {