mirror of
				https://github.com/go-gitea/gitea.git
				synced 2025-10-27 01:24:54 +01:00 
			
		
		
		
	This PR fixes inconsistencies between system and default webhooks in the Gitea API. (See also #26418) - A system webhook is a webhook that captures events for all repositories. - A default webhook is copied to a new repository when it is created. Before this PR `POST /api/v1/admin/hooks/` creates default webhooks (if not configured otherwise) and `GET /api/v1/admin/hooks/` returns system webhooks. The PR introduces an optional query parameter to `GET /api/v1/admin/hooks/` to enable selecting if either default, system or both kind of webhooks should be retrieved. By default the flag is set to return system webhooks keep current behaviour. ## Examples ### System Webhooks #### Create ``` POST /api/v1/admin/hooks/ { "type": "gitea", "active": false, "branch_filter": "*", "events": [ "create", "..." ], "config": { "url": "http://...", "content_type": "json", "secret": "secret", "is_system_webhook": true // <-- controls hook type } } ``` #### List ``` GET/api/v1/admin/hooks?type=system //type argument is optional here since it's the default ``` #### Others The other relevant endpoints work as expected by referencing the hook by id ``` GET /api/v1/admin/hooks/:id PATCH /api/v1/admin/hooks/:id DELETE /api/v1/admin/hooks/:id ``` ### Default Webhooks #### Create ``` POST /api/v1/admin/hooks/ { "type": "gitea", "active": false, "branch_filter": "*", "events": [ "create", "..." ], "config": { "url": "http://...", "content_type": "json", "secret": "secret", "is_system_webhook": false // optional, as false is the default value } } ``` #### List ``` GET/api/v1/admin/hooks?type=default ``` #### Others The other relevant endpoints work as expected by referencing the hook by id ``` GET /api/v1/admin/hooks/:id PATCH /api/v1/admin/hooks/:id DELETE /api/v1/admin/hooks/:id ```
		
			
				
	
	
		
			95 lines
		
	
	
		
			2.8 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			95 lines
		
	
	
		
			2.8 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| // Copyright 2021 The Gitea Authors. All rights reserved.
 | |
| // SPDX-License-Identifier: MIT
 | |
| 
 | |
| package webhook
 | |
| 
 | |
| import (
 | |
| 	"context"
 | |
| 	"fmt"
 | |
| 
 | |
| 	"code.gitea.io/gitea/models/db"
 | |
| 	"code.gitea.io/gitea/modules/optional"
 | |
| )
 | |
| 
 | |
| // GetSystemOrDefaultWebhooks returns webhooks by given argument or all if argument is missing.
 | |
| func GetSystemOrDefaultWebhooks(ctx context.Context, isSystemWebhook optional.Option[bool]) ([]*Webhook, error) {
 | |
| 	webhooks := make([]*Webhook, 0, 5)
 | |
| 	if !isSystemWebhook.Has() {
 | |
| 		return webhooks, db.GetEngine(ctx).Where("repo_id=? AND owner_id=?", 0, 0).
 | |
| 			Find(&webhooks)
 | |
| 	}
 | |
| 
 | |
| 	return webhooks, db.GetEngine(ctx).
 | |
| 		Where("repo_id=? AND owner_id=? AND is_system_webhook=?", 0, 0, isSystemWebhook.Value()).
 | |
| 		Find(&webhooks)
 | |
| }
 | |
| 
 | |
| // GetDefaultWebhooks returns all admin-default webhooks.
 | |
| func GetDefaultWebhooks(ctx context.Context) ([]*Webhook, error) {
 | |
| 	webhooks := make([]*Webhook, 0, 5)
 | |
| 	return webhooks, db.GetEngine(ctx).
 | |
| 		Where("repo_id=? AND owner_id=? AND is_system_webhook=?", 0, 0, false).
 | |
| 		Find(&webhooks)
 | |
| }
 | |
| 
 | |
| // GetSystemOrDefaultWebhook returns admin system or default webhook by given ID.
 | |
| func GetSystemOrDefaultWebhook(ctx context.Context, id int64) (*Webhook, error) {
 | |
| 	webhook := &Webhook{ID: id}
 | |
| 	has, err := db.GetEngine(ctx).
 | |
| 		Where("repo_id=? AND owner_id=?", 0, 0).
 | |
| 		Get(webhook)
 | |
| 	if err != nil {
 | |
| 		return nil, err
 | |
| 	} else if !has {
 | |
| 		return nil, ErrWebhookNotExist{ID: id}
 | |
| 	}
 | |
| 	return webhook, nil
 | |
| }
 | |
| 
 | |
| // GetSystemWebhooks returns all admin system webhooks.
 | |
| func GetSystemWebhooks(ctx context.Context, isActive optional.Option[bool]) ([]*Webhook, error) {
 | |
| 	webhooks := make([]*Webhook, 0, 5)
 | |
| 	if !isActive.Has() {
 | |
| 		return webhooks, db.GetEngine(ctx).
 | |
| 			Where("repo_id=? AND owner_id=? AND is_system_webhook=?", 0, 0, true).
 | |
| 			Find(&webhooks)
 | |
| 	}
 | |
| 	return webhooks, db.GetEngine(ctx).
 | |
| 		Where("repo_id=? AND owner_id=? AND is_system_webhook=? AND is_active = ?", 0, 0, true, isActive.Value()).
 | |
| 		Find(&webhooks)
 | |
| }
 | |
| 
 | |
| // DeleteDefaultSystemWebhook deletes an admin-configured default or system webhook (where Org and Repo ID both 0)
 | |
| func DeleteDefaultSystemWebhook(ctx context.Context, id int64) error {
 | |
| 	return db.WithTx(ctx, func(ctx context.Context) error {
 | |
| 		count, err := db.GetEngine(ctx).
 | |
| 			Where("repo_id=? AND owner_id=?", 0, 0).
 | |
| 			Delete(&Webhook{ID: id})
 | |
| 		if err != nil {
 | |
| 			return err
 | |
| 		} else if count == 0 {
 | |
| 			return ErrWebhookNotExist{ID: id}
 | |
| 		}
 | |
| 
 | |
| 		_, err = db.DeleteByBean(ctx, &HookTask{HookID: id})
 | |
| 		return err
 | |
| 	})
 | |
| }
 | |
| 
 | |
| // CopyDefaultWebhooksToRepo creates copies of the default webhooks in a new repo
 | |
| func CopyDefaultWebhooksToRepo(ctx context.Context, repoID int64) error {
 | |
| 	ws, err := GetDefaultWebhooks(ctx)
 | |
| 	if err != nil {
 | |
| 		return fmt.Errorf("GetDefaultWebhooks: %v", err)
 | |
| 	}
 | |
| 
 | |
| 	for _, w := range ws {
 | |
| 		w.ID = 0
 | |
| 		w.RepoID = repoID
 | |
| 		if err := CreateWebhook(ctx, w); err != nil {
 | |
| 			return fmt.Errorf("CreateWebhook: %v", err)
 | |
| 		}
 | |
| 	}
 | |
| 	return nil
 | |
| }
 |