fix(notifications): support gitlab nested projects for FullNameByRepoURL

Signed-off-by: Zadkiel Aharonian <[email protected]>
aslafy-z · Dec 5, 2023 · 71b79d2 · 71b79d2
1 parent 7484f1d
commit 71b79d2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/docs/operator-manual/notifications/functions.md b/docs/operator-manual/notifications/functions.md
@@ -46,7 +46,7 @@ Transforms given GIT URL into HTTPs format.
 <hr>
 **`repo.FullNameByRepoURL(url string) string`**
 
-Returns repository URL full name `(<owner>/<repoName>)`. Currently supports only Github, GitLab and Bitbucket.
+Returns repository URL full name `(<group>/[<nestedGroup...>/]<repoName>)`. Currently supports only Github, GitLab and Bitbucket.
 
 <hr>
 **`repo.GetCommitMetadata(sha string) CommitMetadata`**

diff --git a/util/notification/expression/repo/repo.go b/util/notification/expression/repo/repo.go
@@ -69,11 +69,8 @@ func FullNameByRepoURL(rawURL string) string {
 	}
 
 	path := gitSuffix.ReplaceAllString(parsed.Path, "")
-	if pathParts := text.SplitRemoveEmpty(path, "/"); len(pathParts) >= 2 {
-		return strings.Join(pathParts[:2], "/")
-	}
-
-	return path
+	pathParts := text.SplitRemoveEmpty(path, "/")
+	return strings.Join(pathParts, "/")
 }
 
 func repoURLToHTTPS(rawURL string) string {