diff options
Diffstat (limited to 'cmd')
-rw-r--r-- | cmd/hook.go | 35 | ||||
-rw-r--r-- | cmd/serv.go | 3 |
2 files changed, 18 insertions, 20 deletions
diff --git a/cmd/hook.go b/cmd/hook.go index ec60ac05..d26334f2 100644 --- a/cmd/hook.go +++ b/cmd/hook.go @@ -27,7 +27,6 @@ import ( "github.com/gogs/gogs/pkg/mailer" "github.com/gogs/gogs/pkg/setting" "github.com/gogs/gogs/pkg/template" - http "github.com/gogs/gogs/routes/repo" ) var ( @@ -71,7 +70,7 @@ func runHookPreReceive(c *cli.Context) error { } setup(c, "hooks/pre-receive.log", true) - isWiki := strings.Contains(os.Getenv(http.ENV_REPO_CUSTOM_HOOKS_PATH), ".wiki.git/") + isWiki := strings.Contains(os.Getenv(models.ENV_REPO_CUSTOM_HOOKS_PATH), ".wiki.git/") buf := bytes.NewBuffer(nil) scanner := bufio.NewScanner(os.Stdin) @@ -92,7 +91,7 @@ func runHookPreReceive(c *cli.Context) error { branchName := strings.TrimPrefix(string(fields[2]), git.BRANCH_PREFIX) // Branch protection - repoID := com.StrTo(os.Getenv(http.ENV_REPO_ID)).MustInt64() + repoID := com.StrTo(os.Getenv(models.ENV_REPO_ID)).MustInt64() protectBranch, err := models.GetProtectBranchOfRepoByName(repoID, branchName) if err != nil { if errors.IsErrBranchNotExist(err) { @@ -108,7 +107,7 @@ func runHookPreReceive(c *cli.Context) error { bypassRequirePullRequest := false // Check if user is in whitelist when enabled - userID := com.StrTo(os.Getenv(http.ENV_AUTH_USER_ID)).MustInt64() + userID := com.StrTo(os.Getenv(models.ENV_AUTH_USER_ID)).MustInt64() if protectBranch.EnableWhitelist { if !models.IsUserInProtectBranchWhitelist(repoID, userID, branchName) { fail(fmt.Sprintf("Branch '%s' is protected and you are not in the push whitelist", branchName), "") @@ -129,7 +128,7 @@ func runHookPreReceive(c *cli.Context) error { // Check force push output, err := git.NewCommand("rev-list", "--max-count=1", oldCommitID, "^"+newCommitID). - RunInDir(models.RepoPath(os.Getenv(http.ENV_REPO_OWNER_NAME), os.Getenv(http.ENV_REPO_NAME))) + RunInDir(models.RepoPath(os.Getenv(models.ENV_REPO_OWNER_NAME), os.Getenv(models.ENV_REPO_NAME))) if err != nil { fail("Internal error", "Fail to detect force push: %v", err) } else if len(output) > 0 { @@ -137,7 +136,7 @@ func runHookPreReceive(c *cli.Context) error { } } - customHooksPath := filepath.Join(os.Getenv(http.ENV_REPO_CUSTOM_HOOKS_PATH), "pre-receive") + customHooksPath := filepath.Join(os.Getenv(models.ENV_REPO_CUSTOM_HOOKS_PATH), "pre-receive") if !com.IsFile(customHooksPath) { return nil } @@ -148,7 +147,7 @@ func runHookPreReceive(c *cli.Context) error { } else { hookCmd = exec.Command(customHooksPath) } - hookCmd.Dir = models.RepoPath(os.Getenv(http.ENV_REPO_OWNER_NAME), os.Getenv(http.ENV_REPO_NAME)) + hookCmd.Dir = models.RepoPath(os.Getenv(models.ENV_REPO_OWNER_NAME), os.Getenv(models.ENV_REPO_NAME)) hookCmd.Stdout = os.Stdout hookCmd.Stdin = buf hookCmd.Stderr = os.Stderr @@ -171,7 +170,7 @@ func runHookUpdate(c *cli.Context) error { fail("First argument 'refName' is empty", "First argument 'refName' is empty") } - customHooksPath := filepath.Join(os.Getenv(http.ENV_REPO_CUSTOM_HOOKS_PATH), "update") + customHooksPath := filepath.Join(os.Getenv(models.ENV_REPO_CUSTOM_HOOKS_PATH), "update") if !com.IsFile(customHooksPath) { return nil } @@ -182,7 +181,7 @@ func runHookUpdate(c *cli.Context) error { } else { hookCmd = exec.Command(customHooksPath, args...) } - hookCmd.Dir = models.RepoPath(os.Getenv(http.ENV_REPO_OWNER_NAME), os.Getenv(http.ENV_REPO_NAME)) + hookCmd.Dir = models.RepoPath(os.Getenv(models.ENV_REPO_OWNER_NAME), os.Getenv(models.ENV_REPO_NAME)) hookCmd.Stdout = os.Stdout hookCmd.Stdin = os.Stdin hookCmd.Stderr = os.Stderr @@ -205,7 +204,7 @@ func runHookPostReceive(c *cli.Context) error { mailer.InitMailRender(path.Join(setting.StaticRootPath, "templates/mail"), path.Join(setting.CustomPath, "templates/mail"), template.NewFuncMap()) - isWiki := strings.Contains(os.Getenv(http.ENV_REPO_CUSTOM_HOOKS_PATH), ".wiki.git/") + isWiki := strings.Contains(os.Getenv(models.ENV_REPO_CUSTOM_HOOKS_PATH), ".wiki.git/") buf := bytes.NewBuffer(nil) scanner := bufio.NewScanner(os.Stdin) @@ -227,10 +226,10 @@ func runHookPostReceive(c *cli.Context) error { OldCommitID: string(fields[0]), NewCommitID: string(fields[1]), RefFullName: string(fields[2]), - PusherID: com.StrTo(os.Getenv(http.ENV_AUTH_USER_ID)).MustInt64(), - PusherName: os.Getenv(http.ENV_AUTH_USER_NAME), - RepoUserName: os.Getenv(http.ENV_REPO_OWNER_NAME), - RepoName: os.Getenv(http.ENV_REPO_NAME), + PusherID: com.StrTo(os.Getenv(models.ENV_AUTH_USER_ID)).MustInt64(), + PusherName: os.Getenv(models.ENV_AUTH_USER_NAME), + RepoUserName: os.Getenv(models.ENV_REPO_OWNER_NAME), + RepoName: os.Getenv(models.ENV_REPO_NAME), } if err := models.PushUpdate(options); err != nil { log.Error(2, "PushUpdate: %v", err) @@ -239,8 +238,8 @@ func runHookPostReceive(c *cli.Context) error { // Ask for running deliver hook and test pull request tasks reqURL := setting.LocalURL + options.RepoUserName + "/" + options.RepoName + "/tasks/trigger?branch=" + template.EscapePound(strings.TrimPrefix(options.RefFullName, git.BRANCH_PREFIX)) + - "&secret=" + os.Getenv(http.ENV_REPO_OWNER_SALT_MD5) + - "&pusher=" + os.Getenv(http.ENV_AUTH_USER_ID) + "&secret=" + os.Getenv(models.ENV_REPO_OWNER_SALT_MD5) + + "&pusher=" + os.Getenv(models.ENV_AUTH_USER_ID) log.Trace("Trigger task: %s", reqURL) resp, err := httplib.Head(reqURL).SetTLSClientConfig(&tls.Config{ @@ -256,7 +255,7 @@ func runHookPostReceive(c *cli.Context) error { } } - customHooksPath := filepath.Join(os.Getenv(http.ENV_REPO_CUSTOM_HOOKS_PATH), "post-receive") + customHooksPath := filepath.Join(os.Getenv(models.ENV_REPO_CUSTOM_HOOKS_PATH), "post-receive") if !com.IsFile(customHooksPath) { return nil } @@ -267,7 +266,7 @@ func runHookPostReceive(c *cli.Context) error { } else { hookCmd = exec.Command(customHooksPath) } - hookCmd.Dir = models.RepoPath(os.Getenv(http.ENV_REPO_OWNER_NAME), os.Getenv(http.ENV_REPO_NAME)) + hookCmd.Dir = models.RepoPath(os.Getenv(models.ENV_REPO_OWNER_NAME), os.Getenv(models.ENV_REPO_NAME)) hookCmd.Stdout = os.Stdout hookCmd.Stdin = buf hookCmd.Stderr = os.Stderr diff --git a/cmd/serv.go b/cmd/serv.go index e34e8b10..50d73f1b 100644 --- a/cmd/serv.go +++ b/cmd/serv.go @@ -19,7 +19,6 @@ import ( "github.com/gogs/gogs/models" "github.com/gogs/gogs/models/errors" "github.com/gogs/gogs/pkg/setting" - http "github.com/gogs/gogs/routes/repo" ) const ( @@ -252,7 +251,7 @@ func runServ(c *cli.Context) error { gitCmd = exec.Command(verb, repoFullName) } if requestMode == models.ACCESS_MODE_WRITE { - gitCmd.Env = append(os.Environ(), http.ComposeHookEnvs(http.ComposeHookEnvsOptions{ + gitCmd.Env = append(os.Environ(), models.ComposeHookEnvs(models.ComposeHookEnvsOptions{ AuthUser: user, OwnerName: owner.Name, OwnerSalt: owner.Salt, |