diff --git a/cmd/hook.go b/cmd/hook.go index 15ed1eb7f..2e0b2ab3b 100644 --- a/cmd/hook.go +++ b/cmd/hook.go @@ -488,10 +488,11 @@ func pushOptions() map[string]string { if pushCount, err := strconv.Atoi(os.Getenv(private.GitPushOptionCount)); err == nil { for idx := 0; idx < pushCount; idx++ { opt := os.Getenv(fmt.Sprintf("GIT_PUSH_OPTION_%d", idx)) - kv := strings.SplitN(opt, "=", 2) - if len(kv) == 2 { - opts[kv[0]] = kv[1] + key, value, found := strings.Cut(opt, "=") + if !found { + value = "true" } + opts[key] = value } } return opts @@ -631,10 +632,11 @@ Forgejo or set your environment appropriately.`, "") break } - kv := strings.SplitN(string(rs.Data), "=", 2) - if len(kv) == 2 { - hookOptions.GitPushOptions[kv[0]] = kv[1] + key, value, found := strings.Cut(string(rs.Data), "=") + if !found { + value = "true" } + hookOptions.GitPushOptions[key] = value } } diff --git a/cmd/hook_test.go b/cmd/hook_test.go index 0c1bee29f..89dafeaa5 100644 --- a/cmd/hook_test.go +++ b/cmd/hook_test.go @@ -15,6 +15,7 @@ import ( "testing" "time" + "code.gitea.io/gitea/modules/private" "code.gitea.io/gitea/modules/setting" "code.gitea.io/gitea/modules/test" @@ -162,3 +163,17 @@ func TestDelayWriter(t *testing.T) { require.Empty(t, out) }) } + +func TestPushOptions(t *testing.T) { + require.NoError(t, os.Setenv(private.GitPushOptionCount, "3")) + require.NoError(t, os.Setenv("GIT_PUSH_OPTION_0", "force-push")) + require.NoError(t, os.Setenv("GIT_PUSH_OPTION_1", "option=value")) + require.NoError(t, os.Setenv("GIT_PUSH_OPTION_2", "option-double=another=value")) + require.NoError(t, os.Setenv("GIT_PUSH_OPTION_3", "not=valid")) + + assert.Equal(t, map[string]string{ + "force-push": "true", + "option": "value", + "option-double": "another=value", + }, pushOptions()) +} diff --git a/tests/integration/git_test.go b/tests/integration/git_test.go index 782390002..708f9a820 100644 --- a/tests/integration/git_test.go +++ b/tests/integration/git_test.go @@ -1025,7 +1025,7 @@ func doCreateAgitFlowPull(dstPath string, ctx *APITestContext, baseBranch, headB t.Run("Succeeds", func(t *testing.T) { defer tests.PrintCurrentTest(t)() - _, _, gitErr := git.NewCommand(git.DefaultContext, "push", "origin", "-o", "force-push=true").AddDynamicArguments("HEAD:refs/for/master/" + headBranch + "-force-push").RunStdString(&git.RunOpts{Dir: dstPath}) + _, _, gitErr := git.NewCommand(git.DefaultContext, "push", "origin", "-o", "force-push").AddDynamicArguments("HEAD:refs/for/master/" + headBranch + "-force-push").RunStdString(&git.RunOpts{Dir: dstPath}) assert.NoError(t, gitErr) currentHeadCommitID, err := upstreamGitRepo.GetRefCommitID(pr.GetGitRefName())