diff options
author | Sami Hiltunen <shiltunen@gitlab.com> | 2023-07-10 23:01:28 +0300 |
---|---|---|
committer | Sami Hiltunen <shiltunen@gitlab.com> | 2023-07-11 11:10:52 +0300 |
commit | fa5a5186d74744e9bb62d13d82323f0c02956e1c (patch) | |
tree | 7cf4abd8bb93746b1309acdab5547c2b7005856a /internal/gitaly/server/auth_test.go | |
parent | 1a3c14aaac2b3fa39f9ad9f0f407fbb0ea9469a0 (diff) |
Move UpdaterWithHooks from git to hook package
UpdaterWithHooks wraps 'git update-ref' with executing the hooks by
calling Gitaly's HookManager. While the Updater itself is just a plain
'update-ref' wrapper, UpdaterWithHooks ties more into Gitaly's domain
types. This is prone to creating cyclic dependencies as anything that
needs to use the 'update-ref' wrapper will also import the hook package
from Gitaly. This creates a problem for TransactionManager as we are
about to integrate some of the transaction logic in the hooks. Fix
the cyclic dependency by moving UpdaterWithHooks into its own subpackage
below the main 'hook' package.
Diffstat (limited to 'internal/gitaly/server/auth_test.go')
-rw-r--r-- | internal/gitaly/server/auth_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/internal/gitaly/server/auth_test.go b/internal/gitaly/server/auth_test.go index 2e83b9a34..d275319f5 100644 --- a/internal/gitaly/server/auth_test.go +++ b/internal/gitaly/server/auth_test.go @@ -17,10 +17,10 @@ import ( "gitlab.com/gitlab-org/gitaly/v16/internal/cache" "gitlab.com/gitlab-org/gitaly/v16/internal/git/catfile" "gitlab.com/gitlab-org/gitaly/v16/internal/git/gittest" - "gitlab.com/gitlab-org/gitaly/v16/internal/git/updateref" "gitlab.com/gitlab-org/gitaly/v16/internal/gitaly/config" "gitlab.com/gitlab-org/gitaly/v16/internal/gitaly/config/auth" "gitlab.com/gitlab-org/gitaly/v16/internal/gitaly/hook" + "gitlab.com/gitlab-org/gitaly/v16/internal/gitaly/hook/updateref" "gitlab.com/gitlab-org/gitaly/v16/internal/gitaly/service" "gitlab.com/gitlab-org/gitaly/v16/internal/gitaly/service/setup" "gitlab.com/gitlab-org/gitaly/v16/internal/gitaly/transaction" |