Skip to content

System-wide webhooks #10546

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Mar 8, 2020
Merged

System-wide webhooks #10546

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
09dbd6b
Create system webhook column (and migration)
jamesorlakin Feb 29, 2020
1decbb5
Create system webhook DB methods
jamesorlakin Feb 29, 2020
c35b7ac
Modify router to handle system webhooks and default ones
jamesorlakin Feb 29, 2020
191a20a
Remove old unused admin nav template
jamesorlakin Feb 29, 2020
47c687c
Adjust orgRepoCtx to differentiate system and default webhook URLs
jamesorlakin Feb 29, 2020
59a2e55
Assign IsSystemWebhook when creating webhooks
jamesorlakin Feb 29, 2020
e03dc4a
Correctly use booleans for IsSystemWebhook
jamesorlakin Feb 29, 2020
d397520
Use system webhooks when preparing webhooks for payload
jamesorlakin Mar 1, 2020
4de3834
Add UI and locale changes
jamesorlakin Mar 1, 2020
af76998
Use router params to differentiate admin hook pages
jamesorlakin Mar 1, 2020
1fa71fa
Fix deleting admin webhooks and rename method
jamesorlakin Mar 1, 2020
cadb482
Add clarity to webhook docs
jamesorlakin Mar 1, 2020
a381e42
Revert "Remove old unused admin nav template"
jamesorlakin Mar 2, 2020
85ede4d
Rename WebHooksNewPost to GiteaHooksNewPost for clarity
jamesorlakin Mar 6, 2020
7695abc
Merge branch 'master' into systemWebhooks
jamesorlakin Mar 6, 2020
0ae62a7
Reintroduce blank line lost during merge conflict
jamesorlakin Mar 6, 2020
94d0246
Merge branch 'master' into systemWebhooks
jamesorlakin Mar 7, 2020
4a6ceb0
Merge branch 'master' into systemWebhooks
lunny Mar 8, 2020
3e63579
Merge branch 'master' into systemWebhooks
lafriks Mar 8, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use router params to differentiate admin hook pages
  • Loading branch information
@jamesorlakin
jamesorlakin committed Mar 1, 2020
commit af76998291ca7d7c289abaf8f45bde9c94681664
4 changes: 1 addition & 3 deletions routers/admin/hooks.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@
package admin

import (
"strings"

"code.gitea.io/gitea/models"
"code.gitea.io/gitea/modules/base"
"code.gitea.io/gitea/modules/context"
Expand All @@ -24,7 +22,7 @@ func DefaultAndSystemWebhooks(ctx *context.Context) {
var err error

// Are we looking at default webhooks?
if strings.Contains(ctx.Link, "/admin/hooks") {
if ctx.Params(":configType") == "hooks" {
ctx.Data["Title"] = ctx.Tr("admin.hooks")
ctx.Data["Description"] = ctx.Tr("admin.hooks.desc")
ctx.Data["PageIsAdminHooks"] = true
Expand Down
2 changes: 1 addition & 1 deletion routers/repo/webhook.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ func getOrgRepoCtx(ctx *context.Context) (*orgRepoCtx, error) {

if ctx.User.IsAdmin {
// Are we looking at default webhooks?
if strings.Contains(ctx.Link, "/admin/hooks") {
if ctx.Params(":configType") == "hooks" {
return &orgRepoCtx{
IsAdmin: true,
Link: path.Join(setting.AppSubURL, "/admin/hooks"),
Expand Down
2 changes: 1 addition & 1 deletion routers/routes/routes.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -453,7 +453,7 @@ func RegisterRoutes(m *macaron.Macaron) {
m.Post("/delete", admin.DeleteRepo)
})

m.Group("/^:type(hooks|system-hooks)$", func() {
m.Group("/^:configType(hooks|system-hooks)$", func() {
m.Get("", admin.DefaultAndSystemWebhooks)
m.Post("/delete", admin.DeleteDefaultWebhook)
m.Get("/:type/new", repo.WebhooksNew)
Expand Down