Skip to content

Commit 8fd8f8b

Browse files
ehl-jfmnsboev
andauthored
Fix build evidence summary URL format by removing redundant build name parameter (#1435) (#1440)
Co-authored-by: Mikhail Sboev <[email protected]>
1 parent 315ef60 commit 8fd8f8b

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

artifactory/utils/commandsummary/evidence_url.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import (
77

88
const (
99
releaseBundleEvidenceFormat = "%sui/artifactory/lifecycle?range=Any+Time&bundleName=%s&repositoryKey=%s&releaseBundleVersion=%s&activeVersionTab=Evidence+Graph"
10-
buildEvidenceFormat = "%sui/builds/%s/%s/%s/Evidence/%s?buildRepo=%s"
10+
buildEvidenceFormat = "%sui/builds/%s/%s/%s/Evidence?buildRepo=%s"
1111
artifactEvidenceFormat = "%sui/repos/tree/Evidence/%s?clearFilter=true"
1212
)
1313

@@ -56,7 +56,6 @@ func generateBuildEvidenceUrl(data EvidenceSummaryData, section summarySection)
5656
url.QueryEscape(data.BuildName),
5757
data.BuildNumber,
5858
data.BuildTimestamp,
59-
url.QueryEscape(data.BuildName),
6059
data.RepoKey)
6160

6261
return addGitHubTrackingToUrl(urlStr, section)

artifactory/utils/commandsummary/evidence_url_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ func TestGenerateEvidenceUrlByType(t *testing.T) {
7373
BuildTimestamp: "1234567890",
7474
RepoKey: "artifactory-build-info",
7575
},
76-
expectedURL: "https://myplatform.com/ui/builds/my-build/123/1234567890/Evidence/my-build?buildRepo=artifactory-build-info&gh_job_id=JFrog+CLI+Core+Tests&gh_section=evidence&m=3&s=1",
76+
expectedURL: "https://myplatform.com/ui/builds/my-build/123/1234567890/Evidence?buildRepo=artifactory-build-info&gh_job_id=JFrog+CLI+Core+Tests&gh_section=evidence&m=3&s=1",
7777
},
7878
{
7979
name: "Build with special characters in name",
@@ -85,7 +85,7 @@ func TestGenerateEvidenceUrlByType(t *testing.T) {
8585
BuildTimestamp: "1234567890",
8686
RepoKey: "artifactory-build-info",
8787
},
88-
expectedURL: "https://myplatform.com/ui/builds/my+build+with+spaces/123/1234567890/Evidence/my+build+with+spaces?buildRepo=artifactory-build-info&gh_job_id=JFrog+CLI+Core+Tests&gh_section=evidence&m=3&s=1",
88+
expectedURL: "https://myplatform.com/ui/builds/my+build+with+spaces/123/1234567890/Evidence?buildRepo=artifactory-build-info&gh_job_id=JFrog+CLI+Core+Tests&gh_section=evidence&m=3&s=1",
8989
},
9090
{
9191
name: "Invalid release bundle falls back to artifact URL",

0 commit comments

Comments
 (0)