From 2b55484c8f0d3d7853cd2d8eef62ccacdfc2e143 Mon Sep 17 00:00:00 2001 From: ipsitapp8 Date: Tue, 14 Apr 2026 12:04:36 +0530 Subject: [PATCH 1/3] fix: add nil guard in auditext manager Create function Signed-off-by: ipsitapp8 --- src/pkg/auditext/manager.go | 3 +++ src/pkg/auditext/manager_test.go | 7 +++++++ 2 files changed, 10 insertions(+) diff --git a/src/pkg/auditext/manager.go b/src/pkg/auditext/manager.go index 93cb70cbeaa..31d6b5ac77c 100644 --- a/src/pkg/auditext/manager.go +++ b/src/pkg/auditext/manager.go @@ -77,6 +77,9 @@ func (m *manager) Get(ctx context.Context, id int64) (*model.AuditLogExt, error) // Create ... func (m *manager) Create(ctx context.Context, audit *model.AuditLogExt) (int64, error) { + if audit == nil { + return 0, nil + } if len(config.AuditLogForwardEndpoint(ctx)) > 0 { auditV1.LogMgr.DefaultLogger(ctx).WithField("operator", audit.Username). WithField("time", audit.OpTime).WithField("resourceType", audit.ResourceType). diff --git a/src/pkg/auditext/manager_test.go b/src/pkg/auditext/manager_test.go index e3438a71e56..087f908ff75 100644 --- a/src/pkg/auditext/manager_test.go +++ b/src/pkg/auditext/manager_test.go @@ -15,6 +15,7 @@ package auditext import ( + "context" "testing" "github.com/stretchr/testify/mock" @@ -84,6 +85,12 @@ func (m *managerTestSuite) TestCreate() { m.Equal(int64(1), id) } +func (m *managerTestSuite) TestCreateNil() { + id, err := m.mgr.Create(context.Background(), nil) + m.Require().Nil(err) + m.Equal(int64(0), id) +} + func (m *managerTestSuite) TestDelete() { m.dao.On("Delete", mock.Anything, mock.Anything).Return(nil) err := m.mgr.Delete(nil, 1) From 29f43a84d5a529890ce36ea1cf0b13c4ff2957e4 Mon Sep 17 00:00:00 2001 From: ipsitapp8 Date: Tue, 21 Apr 2026 13:25:24 +0530 Subject: [PATCH 2/3] fix(ui): prevent tag overlap in artifact list (#23110) Refactored the tag rendering logic in the artifact list to use individual flexbox elements with wrapping instead of a single joint string. This prevents visual overlap and layout breaks when numerous tags are present. Signed-off-by: ipsitapp8 --- .../artifact-list-tab.component.html | 13 +++++-------- .../artifact-list-tab.component.scss | 12 ++++++++++++ 2 files changed, 17 insertions(+), 8 deletions(-) diff --git a/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.html b/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.html index 3231851702e..6512e393c26 100644 --- a/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.html +++ b/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.html @@ -317,19 +317,16 @@ -
+
- {{ tagsString(artifact?.tags) }} +
+ {{ tag.name }} +
({{ artifact?.tagNumber }} ) diff --git a/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.scss b/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.scss index c6e1cc8e728..392a8d495a7 100644 --- a/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.scss +++ b/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.scss @@ -24,6 +24,18 @@ overflow: hidden; white-space: nowrap; text-overflow: ellipsis; + flex-shrink: 1; +} + +.tags-wrapper { + display: flex; + flex-wrap: wrap; + gap: 4px; + width: 100%; +} + +.tag-item { + white-space: nowrap; } .right-pos { From ce43d26ea0cf232f59318dd438ed42c38c06b0a8 Mon Sep 17 00:00:00 2001 From: ipsitapp8 Date: Thu, 23 Jul 2026 16:07:57 +0530 Subject: [PATCH 3/3] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: ipsitapp8 --- .../artifact-list-tab/artifact-list-tab.component.scss | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.scss b/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.scss index 082e08c393d..e343531af98 100644 --- a/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.scss +++ b/src/portal/src/app/base/project/repository/artifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.scss @@ -35,7 +35,11 @@ } .tag-item { + min-width: 0; + max-width: 100%; + overflow: hidden; white-space: nowrap; + text-overflow: ellipsis; } .right-pos {