From 6051fa016cb90c8fcb220c75ecd0e1bd67bebead Mon Sep 17 00:00:00 2001 From: Gabriel Adrian Samfira Date: Tue, 15 Aug 2023 21:26:31 +0000 Subject: [PATCH] Return bad request if hook already installed Signed-off-by: Gabriel Adrian Samfira --- runner/pool/organization.go | 2 +- runner/pool/repository.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/runner/pool/organization.go b/runner/pool/organization.go index 911402e2..b97ef94e 100644 --- a/runner/pool/organization.go +++ b/runner/pool/organization.go @@ -273,7 +273,7 @@ func (r *organization) InstallHook(ctx context.Context, req *github.Hook) error for _, hook := range allHooks { if hook.Config["url"] == req.Config["url"] { - return fmt.Errorf("hook already installed: %w", runnerErrors.ErrBadRequest) + return runnerErrors.NewBadRequestError("hook already installed") } } diff --git a/runner/pool/repository.go b/runner/pool/repository.go index 6cb6b301..646b0aab 100644 --- a/runner/pool/repository.go +++ b/runner/pool/repository.go @@ -274,7 +274,7 @@ func (r *repository) InstallHook(ctx context.Context, req *github.Hook) error { for _, hook := range allHooks { if hook.Config["url"] == req.Config["url"] { - return fmt.Errorf("hook already installed: %w", runnerErrors.ErrBadRequest) + return runnerErrors.NewBadRequestError("hook already installed") } }