Skip to content

Commit 4721100

Browse files
committed
use repository apiurl method
1 parent 59cf964 commit 4721100

File tree

2 files changed

+6
-11
lines changed

2 files changed

+6
-11
lines changed

routers/api/v1/repo/action.go

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -936,14 +936,12 @@ func GetArtifactsOfRun(ctx *context.APIContext) {
936936
return
937937
}
938938

939-
repoName := ctx.Repo.Repository.FullName()
940-
941939
res := new(api.ActionArtifactsResponse)
942940
res.TotalCount = total
943941

944942
res.Entries = make([]*api.ActionArtifact, len(artifacts))
945943
for i := range artifacts {
946-
convertedArtifact, err := convert.ToActionArtifact(ctx, repoName, artifacts[i])
944+
convertedArtifact, err := convert.ToActionArtifact(ctx, ctx.Repo.Repository, artifacts[i])
947945
if err != nil {
948946
ctx.Error(http.StatusInternalServerError, "ToActionArtifact", err)
949947
return
@@ -999,14 +997,12 @@ func GetArtifacts(ctx *context.APIContext) {
999997
return
1000998
}
1001999

1002-
repoName := ctx.Repo.Repository.FullName()
1003-
10041000
res := new(api.ActionArtifactsResponse)
10051001
res.TotalCount = total
10061002

10071003
res.Entries = make([]*api.ActionArtifact, len(artifacts))
10081004
for i := range artifacts {
1009-
convertedArtifact, err := convert.ToActionArtifact(ctx, repoName, artifacts[i])
1005+
convertedArtifact, err := convert.ToActionArtifact(ctx, ctx.Repo.Repository, artifacts[i])
10101006
if err != nil {
10111007
ctx.Error(http.StatusInternalServerError, "ToActionArtifact", err)
10121008
return
@@ -1054,8 +1050,7 @@ func GetArtifact(ctx *context.APIContext) {
10541050
}
10551051

10561052
if actions.IsArtifactV4(art) {
1057-
repoName := ctx.Repo.Repository.FullName()
1058-
convertedArtifact, err := convert.ToActionArtifact(ctx, repoName, art)
1053+
convertedArtifact, err := convert.ToActionArtifact(ctx, ctx.Repo.Repository, art)
10591054
if err != nil {
10601055
ctx.Error(http.StatusInternalServerError, "ToActionArtifact", err)
10611056
return

services/convert/convert.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,12 @@ import (
1919
"code.gitea.io/gitea/models/organization"
2020
"code.gitea.io/gitea/models/perm"
2121
access_model "code.gitea.io/gitea/models/perm/access"
22+
"code.gitea.io/gitea/models/repo"
2223
repo_model "code.gitea.io/gitea/models/repo"
2324
"code.gitea.io/gitea/models/unit"
2425
user_model "code.gitea.io/gitea/models/user"
2526
"code.gitea.io/gitea/modules/container"
2627
"code.gitea.io/gitea/modules/git"
27-
"code.gitea.io/gitea/modules/httplib"
2828
"code.gitea.io/gitea/modules/log"
2929
"code.gitea.io/gitea/modules/setting"
3030
api "code.gitea.io/gitea/modules/structs"
@@ -231,8 +231,8 @@ func ToActionTask(ctx context.Context, t *actions_model.ActionTask) (*api.Action
231231
}
232232

233233
// ToActionArtifact convert a actions_model.ActionArtifact to an api.ActionArtifact
234-
func ToActionArtifact(ctx context.Context, repoName string, art *actions_model.ActionArtifact) (*api.ActionArtifact, error) {
235-
url := httplib.MakeAbsoluteURL(ctx, setting.AppSubURL+"/api/v1/repos/"+repoName+"/actions/artifacts/"+fmt.Sprintf("%d", art.ID))
234+
func ToActionArtifact(ctx context.Context, repo *repo.Repository, art *actions_model.ActionArtifact) (*api.ActionArtifact, error) {
235+
url := fmt.Sprintf("%s/actions/artifacts/%d", repo.APIURL(), art.ID)
236236

237237
return &api.ActionArtifact{
238238
ID: art.ID,

0 commit comments

Comments
 (0)