luckyimnotanigger/forgejo
1
0
Fork 0
mirror of https://codeberg.org/forgejo/forgejo.git synced 2025-05-31 11:52:10 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

fix: use url.JoinPath to join url parts

Browse source 
This avoids duplicated or more slashes.

fixes #4759
This commit is contained in:
Michael Kriese 2024-07-31 13:55:14 +02:00
parent 49eeec269a
commit 46357e7856
No known key found for this signature in database
GPG key ID: F8D7748549A5986A
2 changed files with 8 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

7
services/pull/merge.go
View file

@ -7,6 +7,7 @@ package pull
import (
"context"
"fmt"
"net/url"
"os"
"path/filepath"
"regexp"
@ -56,7 +57,11 @@ func getMergeMessage(ctx context.Context, baseGitRepo *git.Repository, pr *issue
issueReference = "!"
}
reviewedOn := fmt.Sprintf("Reviewed-on: %s/%s", setting.AppURL, pr.Issue.Link())
issueURL, err := url.JoinPath(setting.AppURL, pr.Issue.Link())
if err != nil {
return "", "", err
}
reviewedOn := fmt.Sprintf("Reviewed-on: %s", issueURL)
reviewedBy := pr.GetApprovers(ctx)
if mergeStyle != "" {