[PATCH v3 3/6] refs: allow skipping the reference-transaction hook
- From
Patrick Steinhardt <ps@pks.im>
- Date
- Jan 13, 2022, 06:11 UTC
- Message-ID
- <be826bae3b564fe003778ea10cb2e06fc753c21d.1642054003.git.ps@pks.im>
- In-Reply-To
- <cover.1642054003.git.ps@pks.im>
The reference-transaction hook is executing whenever we prepare, commit or abort a reference transaction. While this is mostly intentional, in case of the files backend we're leaking the implementation detail that the store is in fact a composite store with one loose and one packed backend to the caller. So while we want to execute the hook for all logical updates, executing it for such implementation details is unexpected.
Prepare for a fix by adding a new flag which allows to skip execution of the hook.
Signed-off-by: Patrick Steinhardt <ps@pks.im> --- refs.c | 3 +++ refs.h | 5 +++++ 2 files changed, 8 insertions(+)
diff --git a/refs.c b/refs.c index 7415864b62..526bf5ed97 100644 --- a/refs.c +++ b/refs.c @@ -2084,6 +2084,9 @@ static int run_transaction_hook(struct ref_transaction *transaction, const char *hook; int ret = 0, i; + if (transaction->flags & REF_TRANSACTION_SKIP_HOOK) + return 0; + hook = find_hook("reference-transaction"); if (!hook) return ret; diff --git a/refs.h b/refs.h index 31f7bf9642..d4056f9fe2 100644 --- a/refs.h +++ b/refs.h @@ -568,6 +568,11 @@ enum action_on_err { UPDATE_REFS_QUIET_ON_ERR }; +/* + * Skip executing the reference-transaction hook. + */ +#define REF_TRANSACTION_SKIP_HOOK (1 << 0) + /* * Begin a reference transaction. The reference transaction must * be freed by calling ref_transaction_free().
-- 2.34.1