github/drone
5
0
Fork 0
mirror of https://github.com/harness/drone.git synced 2025-05-10 00:11:43 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: [CODE-3440]: add fix for go get in cut out terminated path (#3618)

Browse Source 
* fix: [CODE-3440]: add fix for go get in cut out terminated path
This commit is contained in:
Abhinav Singh 2025-04-02 21:29:22 +00:00 committed by Harness
parent d3261ebc20
commit 6b521d3422
2 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/api/middleware/encode/encode.go
View File

@ -270,6 +270,10 @@ func regexPathTerminatedWithMarker(
// case 2: xxxxxxxxxxxxxxxxxxxxxxxxM/ (M is at the end and it should have a / after it) // case 2: xxxxxxxxxxxxxxxxxxxxxxxxM/ (M is at the end and it should have a / after it)
// case 3: xxxxxxxxxxxxxxxxxxxxxxxxxM (M is at the end). // case 3: xxxxxxxxxxxxxxxxxxxxxxxxxM (M is at the end).
func cutOutTerminatedPath(subPath string, marker string) (string, bool) { func cutOutTerminatedPath(subPath string, marker string) (string, bool) {
// base case of marker being empty
if marker == "" {
return subPath, true
}
endsWithSlash := strings.HasSuffix(marker, "/") endsWithSlash := strings.HasSuffix(marker, "/")
if !endsWithSlash { if !endsWithSlash {
marker += "/" marker += "/"

7
app/api/middleware/encode/encode_test.go
View File

@ -252,6 +252,13 @@ func TestCutOutTerminatedPath(t *testing.T) {
expectedPath: "", expectedPath: "",
expectedFound: true, expectedFound: true,
}, },
{
name: "empty marker",
subPath: "/foo/bar",
marker: "",
expectedPath: "/foo/bar",
expectedFound: true,
},
} }
for _, tt := range tests { for _, tt := range tests {