]> source.dussan.org Git - gitea.git/commitdiff
add organization-level webhooks
authorChristopher Brickley <brickley@gmail.com>
Thu, 4 Sep 2014 11:17:00 +0000 (07:17 -0400)
committerChristopher Brickley <brickley@gmail.com>
Fri, 5 Sep 2014 11:19:34 +0000 (07:19 -0400)
12 files changed:
cmd/web.go
conf/locale/locale_en-US.ini
models/action.go
models/webhook.go
public/ng/js/gogs.js
routers/org/setting.go
routers/repo/setting.go
templates/org/settings/hook_new.tmpl [new file with mode: 0644]
templates/org/settings/hooks.tmpl [new file with mode: 0644]
templates/org/settings/nav.tmpl
templates/repo/settings/hook_gogs.tmpl
templates/repo/settings/hook_slack.tmpl

index 57164683ac1cc9d2b590c225e11714448951c375..f7b8d92128dea26b19324dcedc3950f87bf34b5b 100644 (file)
@@ -260,6 +260,13 @@ func runWeb(*cli.Context) {
                        m.Group("/settings", func(r *macaron.Router) {
                                r.Get("", org.Settings)
                                r.Post("", bindIgnErr(auth.UpdateOrgSettingForm{}), org.SettingsPost)
+                               r.Get("/hooks", org.SettingsHooks)
+                               r.Get("/hooks/new", repo.WebHooksNew)
+                               r.Post("/hooks/gogs/new", bindIgnErr(auth.NewWebhookForm{}), repo.WebHooksNewPost)
+                               r.Post("/hooks/slack/new", bindIgnErr(auth.NewSlackHookForm{}), repo.SlackHooksNewPost)
+                               r.Get("/hooks/:id", repo.WebHooksEdit)
+                               r.Post("/hooks/gogs/:id", bindIgnErr(auth.NewWebhookForm{}), repo.WebHooksEditPost)
+                               r.Post("/hooks/slack/:id", bindIgnErr(auth.NewSlackHookForm{}), repo.SlackHooksEditPost)
                                r.Route("/delete", "GET,POST", org.SettingsDelete)
                        })
 
index 4f1acdcda9160336644bbed8d5b6586979cd5a7e..3969074eee36eca42517aea52826f5e7b7a57185 100644 (file)
@@ -270,6 +270,7 @@ settings.delete = Delete Organization
 settings.delete_account = Delete This Organization
 settings.delete_prompt = The operation will delete this organization permanently, and <strong>CANNOT</strong> be undone!
 settings.confirm_delete_account = Confirm Deletion
+settings.hooks_desc = Add webhooks that will be triggered for <strong>all repositories</strong> under this organization.
 
 members.public = Public
 members.public_helper = make private
index f739fc353d346b0753d05877e72a21b555e94ecb..c0992dba68a89c3351be1f8a6f0b8efd5581e7fa 100644 (file)
@@ -220,8 +220,20 @@ func CommitRepoAction(userId, repoUserId int64, userName, actEmail string,
 
        ws, err := GetActiveWebhooksByRepoId(repoId)
        if err != nil {
-               return errors.New("action.CommitRepoAction(GetWebhooksByRepoId): " + err.Error())
-       } else if len(ws) == 0 {
+               return errors.New("action.CommitRepoAction(GetActiveWebhooksByRepoId): " + err.Error())
+       }
+
+       // check if repo belongs to org and append additional webhooks
+       if repo.Owner.IsOrganization() {
+               // get hooks for org
+               orgws, err := GetActiveWebhooksByOrgId(repo.OwnerId)
+               if err != nil {
+                       return errors.New("action.CommitRepoAction(GetActiveWebhooksByOrgId): " + err.Error())
+               }
+               ws = append(ws, orgws...)
+       }
+
+       if len(ws) == 0 {
                return nil
        }
 
index 0b7b3a9948d516453a0052d55b1a2da35036af37..5acc83f59279c14b043305b32580cd302eaedfe6 100644 (file)
@@ -45,6 +45,7 @@ type Webhook struct {
        IsActive     bool
        HookTaskType HookTaskType
        Meta         string `xorm:"TEXT"` // store hook-specific attributes
+       OrgId        int64
 }
 
 // GetEvent handles conversion from Events to HookEvent.
@@ -120,6 +121,18 @@ func DeleteWebhook(hookId int64) error {
        return err
 }
 
+// GetWebhooksByOrgId returns all webhooks for an organization.
+func GetWebhooksByOrgId(orgId int64) (ws []*Webhook, err error) {
+       err = x.Find(&ws, &Webhook{OrgId: orgId})
+       return ws, err
+}
+
+// GetActiveWebhooksByOrgId returns all active webhooks for an organization.
+func GetActiveWebhooksByOrgId(orgId int64) (ws []*Webhook, err error) {
+       err = x.Find(&ws, &Webhook{OrgId: orgId, IsActive: true})
+       return ws, err
+}
+
 //   ___ ___                __   ___________              __
 //  /   |   \  ____   ____ |  | _\__    ___/____    _____|  | __
 // /    ~    \/  _ \ /  _ \|  |/ / |    |  \__  \  /  ___/  |/ /
index c08a887a4cf75db6c97bc5dee0bd112ded054ec8..c60a5cf6480215fede26c44af649d0d4f847e6e2 100644 (file)
@@ -349,17 +349,8 @@ function initRepo() {
     })
 }
 
-function initRepoSetting() {
-    // Options.
-    // Confirmation of changing repository name.
-    $('#repo-setting-form').submit(function (e) {
-        var $reponame = $('#repo_name');
-        if (($reponame.data('repo-name') != $reponame.val()) && !confirm('Repository name has been changed, do you want to continue?')) {
-            e.preventDefault();
-            return true;
-        }
-    });
-
+// when user changes hook type, hide/show proper divs
+function initHookTypeChange() {
     // web hook type change
     $('select#hook-type').on("change", function () {
       hookTypes = ['Gogs','Slack'];
@@ -374,6 +365,20 @@ function initRepoSetting() {
         }
       });
     });
+}
+
+function initRepoSetting() {
+    // Options.
+    // Confirmation of changing repository name.
+    $('#repo-setting-form').submit(function (e) {
+        var $reponame = $('#repo_name');
+        if (($reponame.data('repo-name') != $reponame.val()) && !confirm('Repository name has been changed, do you want to continue?')) {
+            e.preventDefault();
+            return true;
+        }
+    });
+
+    initHookTypeChange();
 
     $('#transfer-button').click(function () {
         $('#transfer-form').show();
@@ -421,6 +426,8 @@ function initOrgSetting() {
             return true;
         }
     });
+
+    initHookTypeChange();
 }
 
 function initInvite() {
index 0ddf0065cce1d7105cfcc36f70bcca5be39c9d15..f853ef0e8e060f3db33d0eec89365bbe30022215 100644 (file)
@@ -5,6 +5,7 @@
 package org
 
 import (
+       "github.com/Unknwon/com"
        "github.com/gogits/gogs/models"
        "github.com/gogits/gogs/modules/auth"
        "github.com/gogits/gogs/modules/base"
@@ -15,6 +16,7 @@ import (
 const (
        SETTINGS_OPTIONS base.TplName = "org/settings/options"
        SETTINGS_DELETE  base.TplName = "org/settings/delete"
+       SETTINGS_HOOKS   base.TplName = "org/settings/hooks"
 )
 
 func Settings(ctx *middleware.Context) {
@@ -97,3 +99,29 @@ func SettingsDelete(ctx *middleware.Context) {
 
        ctx.HTML(200, SETTINGS_DELETE)
 }
+
+func SettingsHooks(ctx *middleware.Context) {
+       ctx.Data["Title"] = ctx.Tr("org.settings")
+       ctx.Data["PageIsSettingsHooks"] = true
+
+       // Delete web hook.
+       remove := com.StrTo(ctx.Query("remove")).MustInt64()
+       if remove > 0 {
+               if err := models.DeleteWebhook(remove); err != nil {
+                       ctx.Handle(500, "DeleteWebhook", err)
+                       return
+               }
+               ctx.Flash.Success(ctx.Tr("repo.settings.remove_hook_success"))
+               ctx.Redirect(ctx.Org.OrgLink + "/settings/hooks")
+               return
+       }
+
+       ws, err := models.GetWebhooksByOrgId(ctx.Org.Organization.Id)
+       if err != nil {
+               ctx.Handle(500, "GetWebhooksByOrgId", err)
+               return
+       }
+
+       ctx.Data["Webhooks"] = ws
+       ctx.HTML(200, SETTINGS_HOOKS)
+}
index 74567812220e7bbea5453f181dfdf73f69569254..81747d43ed04ba98a942d1736c22083ca87f995f 100644 (file)
@@ -6,6 +6,7 @@ package repo
 
 import (
        "encoding/json"
+       "errors"
        "fmt"
        "strings"
        "time"
@@ -26,6 +27,7 @@ const (
        COLLABORATION    base.TplName = "repo/settings/collaboration"
        HOOKS            base.TplName = "repo/settings/hooks"
        HOOK_NEW         base.TplName = "repo/settings/hook_new"
+       ORG_HOOK_NEW     base.TplName = "org/settings/hook_new"
 )
 
 func Settings(ctx *middleware.Context) {
@@ -284,7 +286,14 @@ func WebHooksNew(ctx *middleware.Context) {
        ctx.Data["PageIsSettingsHooksNew"] = true
        ctx.Data["Webhook"] = models.Webhook{HookEvent: &models.HookEvent{}}
        renderHookTypes(ctx)
-       ctx.HTML(200, HOOK_NEW)
+       orgId, repoId, _ := getOrgRepoCtx(ctx)
+       if repoId > 0 {
+               ctx.HTML(200, HOOK_NEW)
+       } else if orgId > 0 {
+               ctx.HTML(200, ORG_HOOK_NEW)
+       } else {
+               ctx.Handle(500, "WebHooksEdit(DetermineContext)", errors.New("Can't determine hook context"))
+       }
 }
 
 func WebHooksNewPost(ctx *middleware.Context, form auth.NewWebhookForm) {
@@ -293,6 +302,8 @@ func WebHooksNewPost(ctx *middleware.Context, form auth.NewWebhookForm) {
        ctx.Data["PageIsSettingsHooksNew"] = true
        ctx.Data["Webhook"] = models.Webhook{HookEvent: &models.HookEvent{}}
 
+       orgId, repoId, link := getOrgRepoCtx(ctx)
+
        if ctx.HasError() {
                ctx.HTML(200, HOOK_NEW)
                return
@@ -304,7 +315,7 @@ func WebHooksNewPost(ctx *middleware.Context, form auth.NewWebhookForm) {
        }
 
        w := &models.Webhook{
-               RepoId:      ctx.Repo.Repository.Id,
+               RepoId:      repoId,
                Url:         form.PayloadUrl,
                ContentType: ct,
                Secret:      form.Secret,
@@ -314,6 +325,7 @@ func WebHooksNewPost(ctx *middleware.Context, form auth.NewWebhookForm) {
                IsActive:     form.Active,
                HookTaskType: models.GOGS,
                Meta:         "",
+               OrgId:        orgId,
        }
 
        if err := w.UpdateEvent(); err != nil {
@@ -325,7 +337,7 @@ func WebHooksNewPost(ctx *middleware.Context, form auth.NewWebhookForm) {
        }
 
        ctx.Flash.Success(ctx.Tr("repo.settings.add_hook_success"))
-       ctx.Redirect(ctx.Repo.RepoLink + "/settings/hooks")
+       ctx.Redirect(link + "/settings/hooks")
 }
 
 func WebHooksEdit(ctx *middleware.Context) {
@@ -363,7 +375,14 @@ func WebHooksEdit(ctx *middleware.Context) {
        }
        w.GetEvent()
        ctx.Data["Webhook"] = w
-       ctx.HTML(200, HOOK_NEW)
+       orgId, repoId, _ := getOrgRepoCtx(ctx)
+       if repoId > 0 {
+               ctx.HTML(200, HOOK_NEW)
+       } else if orgId > 0 {
+               ctx.HTML(200, ORG_HOOK_NEW)
+       } else {
+               ctx.Handle(500, "WebHooksEdit(DetermineContext)", errors.New("Can't determine hook context"))
+       }
 }
 
 func WebHooksEditPost(ctx *middleware.Context, form auth.NewWebhookForm) {
@@ -413,9 +432,10 @@ func WebHooksEditPost(ctx *middleware.Context, form auth.NewWebhookForm) {
                ctx.Handle(500, "WebHooksEditPost", err)
                return
        }
+       _, _, link := getOrgRepoCtx(ctx)
 
        ctx.Flash.Success(ctx.Tr("repo.settings.update_hook_success"))
-       ctx.Redirect(fmt.Sprintf("%s/settings/hooks/%d", ctx.Repo.RepoLink, hookId))
+       ctx.Redirect(fmt.Sprintf("%s/settings/hooks/%d", link, hookId))
 }
 
 func SlackHooksNewPost(ctx *middleware.Context, form auth.NewSlackHookForm) {
@@ -428,6 +448,7 @@ func SlackHooksNewPost(ctx *middleware.Context, form auth.NewSlackHookForm) {
                ctx.HTML(200, HOOK_NEW)
                return
        }
+       orgId, repoId, link := getOrgRepoCtx(ctx)
 
        meta, err := json.Marshal(&models.Slack{
                Domain:  form.Domain,
@@ -440,7 +461,7 @@ func SlackHooksNewPost(ctx *middleware.Context, form auth.NewSlackHookForm) {
        }
 
        w := &models.Webhook{
-               RepoId:      ctx.Repo.Repository.Id,
+               RepoId:      repoId,
                Url:         models.GetSlackURL(form.Domain, form.Token),
                ContentType: models.JSON,
                Secret:      "",
@@ -450,6 +471,7 @@ func SlackHooksNewPost(ctx *middleware.Context, form auth.NewSlackHookForm) {
                IsActive:     form.Active,
                HookTaskType: models.SLACK,
                Meta:         string(meta),
+               OrgId:        orgId,
        }
        if err := w.UpdateEvent(); err != nil {
                ctx.Handle(500, "UpdateEvent", err)
@@ -460,7 +482,7 @@ func SlackHooksNewPost(ctx *middleware.Context, form auth.NewSlackHookForm) {
        }
 
        ctx.Flash.Success(ctx.Tr("repo.settings.add_hook_success"))
-       ctx.Redirect(ctx.Repo.RepoLink + "/settings/hooks")
+       ctx.Redirect(link + "/settings/hooks")
 }
 
 func SlackHooksEditPost(ctx *middleware.Context, form auth.NewSlackHookForm) {
@@ -514,7 +536,25 @@ func SlackHooksEditPost(ctx *middleware.Context, form auth.NewSlackHookForm) {
                ctx.Handle(500, "SlackHooksEditPost", err)
                return
        }
+       _, _, link := getOrgRepoCtx(ctx)
 
        ctx.Flash.Success(ctx.Tr("repo.settings.update_hook_success"))
-       ctx.Redirect(fmt.Sprintf("%s/settings/hooks/%d", ctx.Repo.RepoLink, hookId))
+       ctx.Redirect(fmt.Sprintf("%s/settings/hooks/%d", link, hookId))
+}
+
+func getOrgRepoCtx(ctx *middleware.Context) (int64, int64, string) {
+       orgId := int64(0)
+       repoId := int64(0)
+       link := ""
+       if _, ok := ctx.Data["RepoLink"]; ok {
+               repoId = ctx.Repo.Repository.Id
+               link = ctx.Repo.RepoLink
+       }
+
+       if _, ok := ctx.Data["OrgLink"]; ok {
+               orgId = ctx.Org.Organization.Id
+               link = ctx.Org.OrgLink
+       }
+
+       return orgId, repoId, link
 }
diff --git a/templates/org/settings/hook_new.tmpl b/templates/org/settings/hook_new.tmpl
new file mode 100644 (file)
index 0000000..6e7ee53
--- /dev/null
@@ -0,0 +1,37 @@
+{{template "ng/base/head" .}}
+{{template "ng/base/header" .}}
+{{template "org/base/header" .}}
+<div id="setting-wrapper" class="main-wrapper">
+    <div id="org-setting" class="container clear">
+        {{template "org/settings/nav" .}}
+        <div class="grid-4-5 left">
+            <div class="setting-content">
+                {{template "ng/base/alert" .}}
+                <div id="setting-content">
+                    <div id="repo-hooks-panel" class="panel panel-radius">
+                        <div class="panel-header">
+                          <strong>{{if .PageIsSettingsHooksNew}}{{.i18n.Tr "repo.settings.add_webhook"}}{{else}}{{.i18n.Tr "repo.settings.update_webhook"}}{{end}}</strong>
+                        </div>
+                        {{template "repo/settings/hook_types" .}}
+                        {{template "repo/settings/hook_gogs" .}}
+                        {{template "repo/settings/hook_slack" .}}
+                    </div>
+                </div>
+                {{if .PageIsSettingsHooksEdit}}
+              <br>
+                <div id="setting-content">
+                    <div id="repo-hooks-history-panel" class="panel panel-radius">
+                        <div class="panel-header">
+                          <strong>{{.i18n.Tr "repo.settings.recent_deliveries"}}</strong>
+                        </div>
+                        <ul class="panel-body setting-list">
+                            <li>Coming soon!</li>
+                        </ul>
+                    </div>
+              </div>
+              {{end}}
+            </div>
+        </div>
+  </div>
+</div>
+{{template "ng/base/footer" .}}
diff --git a/templates/org/settings/hooks.tmpl b/templates/org/settings/hooks.tmpl
new file mode 100644 (file)
index 0000000..713cfeb
--- /dev/null
@@ -0,0 +1,38 @@
+{{template "ng/base/head" .}}
+{{template "ng/base/header" .}}
+{{template "org/base/header" .}}
+<div id="setting-wrapper" class="main-wrapper">
+    <div id="org-setting" class="container clear">
+               {{template "org/settings/nav" .}}
+               <div class="grid-4-5 left">
+                   <div class="setting-content">
+                       {{template "ng/base/alert" .}}
+                       <div id="setting-content">
+                      <div id="repo-hooks-panel" class="panel panel-radius">
+                          <div class="panel-header">
+                              <a class="btn btn-small btn-black btn-header btn-radius right" href="{{.OrgLink}}/settings/hooks/new">{{.i18n.Tr "repo.settings.add_webhook"}}</a>
+                            <strong>{{.i18n.Tr "repo.settings.hooks"}}</strong>
+                          </div>
+                          <ul class="panel-body setting-list">
+                              <li>{{.i18n.Tr "org.settings.hooks_desc" | Str2html}}</li>
+                              {{range .Webhooks}}
+                <li>
+                  {{if .IsActive}}
+                  <span class="left text-success"><i class="octicon octicon-check"></i></span>
+                  {{else}}
+                  <span class="left text-grey"><i class="octicon octicon-primitive-dot"></i></span>
+                  {{end}}
+                  <a class="link" href="{{$.OrgLink}}/settings/hooks/{{.Id}}">{{.Url}}</a>
+                  <a href="{{$.OrgLink}}/settings/hooks?remove={{.Id}}" class="text-red right"><i class="fa fa-times"></i></a>
+                              <a href="{{$.OrgLink}}/settings/hooks/{{.Id}}" class="text-blue right"><i class="fa fa-pencil"></i></a>
+                </li>
+                              {{end}}
+                          </ul>
+                      </div>
+                  </div>
+              </div>
+          </div>
+      </div>
+  </div>
+</div>
+{{template "ng/base/footer" .}}
index 950569d686f6751264ec13b906425f8900964eca..954893c6c223b1a0807ad6a4b9955110872e086a 100644 (file)
@@ -5,7 +5,8 @@
     <div class="panel-body">
         <ul class="menu menu-vertical switching-list grid-1-5 left">
             <li {{if .PageIsSettingsOptions}}class="current"{{end}}><a href="/org/{{.Org.LowerName}}/settings">{{.i18n.Tr "org.settings.options"}}</a></li>
+            <li {{if .PageIsSettingsHooks}}class="current"{{end}}><a href="/org/{{.Org.LowerName}}/settings/hooks">{{.i18n.Tr "repo.settings.hooks"}}</a></li>
             <li {{if .PageIsSettingsDelete}}class="current"{{end}}><a href="/org/{{.Org.LowerName}}/settings/delete">{{.i18n.Tr "org.settings.delete"}}</a></li>
         </ul>
     </div>
-</div>
\ No newline at end of file
+</div>
index 35b5899532774e6b692cb7f201d846d6c2745e79..31a04ce09a4046c1aceb6d9490e34498686c06ef 100644 (file)
@@ -1,5 +1,5 @@
 <div id="gogs" class="{{if (and .PageIsSettingsHooksEdit (not (eq .HookType "Gogs")))}}hidden{{end}}">
-  <form class="form form-align panel-body repo-setting-form" id="repo-setting-form-gogs"  action="{{.RepoLink}}/settings/hooks/gogs/{{if .PageIsSettingsHooksNew}}new{{else}}{{.Webhook.Id}}{{end}}" method="post">
+  <form class="form form-align panel-body repo-setting-form" id="repo-setting-form-gogs" action="{{if .RepoLink}}{{.RepoLink}}{{else if .OrgLink}}{{.OrgLink}}{{end}}/settings/hooks/gogs/{{if .PageIsSettingsHooksNew}}new{{else}}{{.Webhook.Id}}{{end}}" method="post">
     {{.CsrfTokenHtml}}
     <input type="hidden" name="hook_type" value="gogs">
     <div class="text-center panel-desc">{{.i18n.Tr "repo.settings.add_webhook_desc" | Str2html}}</div>
index 50d28e2f5e4f231f8fd46de3ce08be7b27f104a5..ed7a42e6f7669250f64d0e5b7c90ba97b3a23948 100644 (file)
@@ -1,5 +1,5 @@
 <div id="slack" class="{{if or .PageIsSettingsHooksNew (and .PageIsSettingsHooksEdit (not (eq .HookType "Slack")))}}hidden{{end}}">
-  <form class="form form-align panel-body repo-setting-form" id="repo-setting-form-slack" action="{{.RepoLink}}/settings/hooks/slack/{{if .PageIsSettingsHooksNew}}new{{else}}{{.Webhook.Id}}{{end}}" method="post">
+  <form class="form form-align panel-body repo-setting-form" id="repo-setting-form-slack" action="{{if .RepoLink}}{{.RepoLink}}{{else if .OrgLink}}{{.OrgLink}}{{end}}/settings/hooks/slack/{{if .PageIsSettingsHooksNew}}new{{else}}{{.Webhook.Id}}{{end}}" method="post">
     {{.CsrfTokenHtml}}
     <input type="hidden" name="hook_type" value="slack">
     <div class="text-center panel-desc">{{.i18n.Tr "repo.settings.add_slack_hook_desc" | Str2html}}</div>