Merge pull request #1082 from nscuro/issue-1081

DependencyTrack · Nov 10, 2024 · b4a4b1a · b4a4b1a
2 parents 0244383 + f359ed3
commit b4a4b1a
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/views/portfolio/tags/TagList.vue b/src/views/portfolio/tags/TagList.vue
@@ -23,6 +23,8 @@ import TaggedNotificationRuleListModal from '@/views/portfolio/tags/TaggedNotifi
 import TaggedPoliciesListModal from '@/views/portfolio/tags/TaggedPoliciesListModal.vue';
 import TaggedProjectListModal from '@/views/portfolio/tags/TaggedProjectListModal.vue';
 import i18n from '@/i18n';
+import MurmurHash2 from 'imurmurhash';
+
 export default {
   mixins: [bootstrapTableMixin, permissionsMixin, routerMixin],
   components: {
@@ -195,7 +197,7 @@ export default {
         },
         buttonsOrder: ['btnDeleteSelected', 'refresh', 'columns'],
         clickToSelect: true,
-        uniqueId: 'name',
+        uniqueId: 'nameHash',
         search: true,
         showColumns: true,
         showRefresh: true,
@@ -212,6 +214,11 @@ export default {
           refresh: 'fa-refresh',
         },
         responseHandler: function (res, xhr) {
+          for (let tag of res) {
+            // Tag names with special characters can break table rendering.
+            // https://github.com/DependencyTrack/dependency-track/issues/4357
+            tag.nameHash = MurmurHash2(tag.name).result();
+          }
           res.total = xhr.getResponseHeader('X-Total-Count');
           return res;
         },