From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 867F2406818 for ; Thu, 3 Sep 2026 09:28:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788427700; cv=none; b=o1zP4fQFsuJO7tRUE9Xv3J1BIaGK7xiNdO8ryzZYQIWHlCSEPFtgeKz2l4fAvjYi2EEj293s6L37nhOkrOdDD/UrLcCWzrxGUrh/wfylS+McxlfDX7HlVHYqfpA3hgkvxZNy4NKJfGNapJLGCof7DNATmg9oWGG7mcT3RzVkXRo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788427700; c=relaxed/simple; bh=MdLDzZwlQpEMM1c81xS8o5KDeL/ydQTpi61o9e8LQRg=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=mjtuT8gJRt0AUUWV8KfDZneAv2IDaCtg3HyzIPnZcR/xtySUcGk6obygU04MYOGoSB2ZUxMGuLagqhXUHIxBoAMqG7L/4pr7uso03V478ZlGbcqwTErVsuzQpytb9ykiSgVflJKgS62T/rTh5xGcCNGVkugbyhfkHZlzy0MERjQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=fbt2L3xU; arc=none smtp.client-ip=209.85.128.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="fbt2L3xU" Received: by mail-wm1-f52.google.com with SMTP id 5b1f17b1804b1-495437bb891so6096415e9.1 for ; Thu, 03 Sep 2026 02:28:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1788427696; x=1789032496; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=fCJ+E4TukvgltiTmcijN8we6GYx4bdmKLVRwHuoyxA4=; b=fbt2L3xUeTv1gnTsZus8vqfT1tKj58BNCBl+4+PZ8l5vG+Hj6Jd0WYbokuJH5n30mU c3fkdXp3wmmoeNAJoZt+I0R6Tf90X/pKHx93yOy0Ko2D0iAn+wLHmYK7yHjS0R9ZXFcN nR0UwpMbvxeAWfhwS3ANSHpOVEpFKL4g6W8+Zl4qqg7NxLYcvgLleSI5Gjpx1utJVfqq EDjOxeLJHlFRIL3Lw2U0qJ/YmpTerITuKzAQXDyVsUBvg+V3KsyO/4jCzL+IN55nz4AZ ZWv0WV7sh4mw3VYj9NmH3HR4FemfQGNANNtYhpMUBtbI6LGPV7Gr5o9nk6/nEuGa8gt1 9OOg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788427696; x=1789032496; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=fCJ+E4TukvgltiTmcijN8we6GYx4bdmKLVRwHuoyxA4=; b=Yaha+3uXZTI/3zYKd2Hi6Lb+IhzYnJUpKe1jwDKwG7TIz/UZmu0IuaHIF01UF1PKwr sdMPX/Wx92LFO/Nk/2WvgTSjY+WiahKPbYIYloKahVBpSXGH6rnvjLpqqel/bTnFJpLe f1iHed7zrqyUrcdo7RhC/PanJv30P9XwAE1jZwNhSuSj9Ug1nBxe3qTXUjpgCFpYI18X 9ZndBuP+/y/D6ciZmJQ+FvnXbKtifqQ655Dqo4O/x//0bgjgvPjfiwsKfa06FyneQE9k PO1omNyox5Db3wLsjOKshD8fqsKinQAhryBR/CidaeZs788Eej2uzv/nUufRvnloTJN2 GJag== X-Gm-Message-State: AFuF++nE5XhBnb1GTDE/pHPvLaEQAcVFZW02bb598MWbbrZXw62waaBW I07Cq7SSbjMEOd/mJTLpRevtBj8U0Vma9ytJxEJXtoSbbVtoPnjUIAIsR2qF725w X-Gm-Gg: AYBFou1EMZpzBesmlTNW8qD1RzI8TeJL8EsRxuaaTGHz9QGjjeCd0jn7vKmQ0qUs91J LDH5Flc4DqDBA9jXKaR60RkTaGdJLCuYE6FZ+CXTYZn8HyoSwy+cMihQMlnZnwOFg9sEhOrbHoO IRxwC2cGjS3k89SEmqQCAXREjSrR1ue3kKDZNEfJaeDJV+M78l/zT7FjOBR8QfKwfDMwoeLQaz5 y9Qgb8vUwcXIqo+sF4N5tqu6Yts5cgqeMudsyzUjyPw3fYCy/AQaj94OankhEzf04WfOu3wh2zu EyDhIN8gLaZDzVqZiNIZv1ER1MwJNzEoKF/uNfmNs3Ot+nZX43mcvZYdeT5e9Ugtglp6TsrLqYo B8JaEDBKQUNYYcq9Pj8vyLzhoggzz66d349XIw1gU8vqpBLhqUxLgoW7yYtG6aT5WHDKjDjPMLv KqNTkqF+T6yku87pp6uvv/s1enT2FA1ET/AYX0nrJddhtpcVYuQCdOM3GBQC1j6wNqTi4k7Jk4V L3QjqiV1DjblU7mGgOOEf2aEsM= X-Received: by 2002:a05:600c:4e05:b0:493:f783:c46a with SMTP id 5b1f17b1804b1-49cf1586ad3mr22971125e9.6.1788427695371; Thu, 03 Sep 2026 02:28:15 -0700 (PDT) Received: from [127.0.0.2] ([2a02:8109:d906:4e00:56e2:6e6b:ebf7:2587]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-49cee60d8a6sm55553105e9.10.2026.09.03.02.28.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 03 Sep 2026 02:28:14 -0700 (PDT) From: Karthik Nayak Date: Thu, 03 Sep 2026 11:28:01 +0200 Subject: [PATCH v6 4/4] hook: introduce the receive-report hook Precedence: bulk X-Mailing-List: git@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20260903-758-introduce-hook-v6-4-6283b1fb9b1c@gmail.com> References: <20260903-758-introduce-hook-v6-0-6283b1fb9b1c@gmail.com> In-Reply-To: <20260903-758-introduce-hook-v6-0-6283b1fb9b1c@gmail.com> To: git@vger.kernel.org Cc: ps@pks.im, gitster@pobox.com, jltobler@gmail.com, kristofferhaugsbakk@fastmail.com, Phillip Wood , Karthik Nayak X-Mailer: b4 0.15.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=19979; i=karthik.188@gmail.com; h=from:subject:message-id; bh=MdLDzZwlQpEMM1c81xS8o5KDeL/ydQTpi61o9e8LQRg=; b=owJ4nAHtARL+kA0DAAoBPtWfJI5GjH8ByyZiAGqZPajeBl4z3RLrHuB114KYKnqwlCniMZh+P zioMFKKuLFTDokBswQAAQoAHRYhBFfOTH9jdXEPy2XGBj7VnySORox/BQJqmT2oAAoJED7VnySO Rox/SEAL/AqzojdrinZGq1y+dD6m5RlW4nSoumdvpFc/cXR0o8ga+vLvsBepzbyin9cg5OPC8+n v8cms8RS+Upi1EMouMmJr34b7s7pvkmHuGLHEHB6TuP0ZV4Gr0PLh3V3yTN+Vrvjbb2h+y0m6Hq SHWQoEmivBW2lLzHgoUyWJ38ytanqK3GlTol4mFsHNZIpE5ZWBGfAhzY0E8/0VEByVq2NcrRBfq ft/oE7gwUZYdmYGRf4jTfIrds6AHuxpzdbM76yfYKWCG68giXbT/TXimR2+fVp5c+UY/X+UN23i owCjLGo6AZzJvGr8KyWT9B+H38+sOw+xp/uvyB8F/jV9Osf7IVpyMssxwQgA8rsO5npKAsvjYOr OuuJCus7eBOoOr4rQeJuF89LMurmqGsxc4vsDH9M24MoVgxwUHtzHfQMv3hBE0eUv+WcOs2hliO Yq4tDepUSxOCWJqAehHqqGTvXFmJyHMyB1ZLPT6tRGKyVcfq37PWC8OkFcNYX3BbzgPzltfHnc8 nY= X-Developer-Key: i=karthik.188@gmail.com; a=openpgp; fpr=57CE4C7F6375710FCB65C6063ED59F248E468C7F When running 'git-receive-pack(1)', there is no way for the server to intercept and modify the status report before it is sent back to the client. Servers with custom logic may need to transform or gate the report based on the outcome of external logic post reference updates. This is specially needed for our usecase at GitLab where we have custom MVCC logic on top of Git which creates a new version for each push operation. The new version is only committed when certain external operations post reference transaction succeed. So reporting the correct message based on the outcome of these operations is important. The outcome of these operations is only known after `execute_commands()` has returned and before the report is written. There is no point in receive-pack where the server can act on that. We cannot use any of the existing hooks as: - The pre-receive hook runs too early, as we haven't updated references at that point yet and we need to have the full view of all resulting updates (both objects and references). - The update hook is too inefficient as it runs once per reference, and we cannot trivially determine the last update. - The reference-transaction hook is not suited for this. It fires from within `ref_transaction_commit()`, which is before the outcome we need to report is known, so there is no phase at which it could give us the answer. It also does not contain any knowledge regarding the push and cannot communicate with the clients. - The proc-receive hook replaces execute_commands() for references matching 'receive.procReceiveRefs'. We need to gate the report for the push as a whole. - The post-receive and post-update hooks cannot be used as they run too late, at the point where we have already reported success to the client. Introduce a new 'receive-report' hook. The hook receives the complete pkt-line encoded status report on standard input, after all ref updates have been applied to the repository by execute_commands() but before the report is sent to the client. See linkgit:gitprotocol-pack[5] details on the protocol structure. The hook's stdout fully replaces the report sent to the client. receive-pack fully buffers the hook's stdout before acting on the exit status, so the exit code is known before the client receives anything. This gives two distinct behaviors depending on exit status: - Exit 0: the hook's stdout is used as the report. The hook can rewrite 'ok' lines to 'ng' lines to signal per-ref rejection to the client while receive-pack itself exits cleanly. The client marks rejected refs as '[remote rejected]' and exits with a non-zero status if any ref is 'ng'. - Non-zero exit: the hook's stdout is discarded, receive-pack modifies all references to be rejected with a 'receive-report hook failed' error. In both cases, any output the hook writes to standard error is forwarded to the client over the sideband channel and appears as 'remote:' lines on the client terminal. Writing to stderr alone does not affect the push outcome. Reference updates applied by execute_commands() are not rolled back in either failure mode. The hook can cause the client to perceive the push as failed, but cannot undo server-side changes. This creates a divergence that the server cannot resolve: the client leaves its remote-tracking reference at the old value while the update is in fact applied, and a later fetch may reveal the update that the push reported as rejected. The hook is therefore only appropriate for servers which can guarantee that a rejected update is not observable by any reader. In our case the transaction committed by execute_commands() produces a candidate version which is not visible to other readers and is only published once the subsequent operations succeed, so a report of 'ng' corresponds to a version that is discarded rather than published. On a repository where a committed reference update is immediately visible, rejecting a push from this hook would instead leave the pusher with a view that does not match the server. This hook does not use the config-based hook infrastructure, which supports running multiple scripts per hook event. This hook is a bidirectional filter: it receives the report on stdin and writes a modified version to stdout. Running multiple such scripts sequentially would require piping the output of one into the input of the next, which the current hook infrastructure does not support. A single-script design is therefore a natural fit, and is consistent with how 'proc-receive' is structured for the same reason. Helped-by: Patrick Steinhardt Signed-off-by: Karthik Nayak --- Documentation/git-receive-pack.adoc | 9 ++ Documentation/githooks.adoc | 61 ++++++++++ builtin/receive-pack.c | 54 +++++++++ t/meson.build | 1 + t/t5412-receive-report-hook.sh | 224 ++++++++++++++++++++++++++++++++++++ 5 files changed, 349 insertions(+) diff --git a/Documentation/git-receive-pack.adoc b/Documentation/git-receive-pack.adoc index 5806792ba7..ab668ffa0c 100644 --- a/Documentation/git-receive-pack.adoc +++ b/Documentation/git-receive-pack.adoc @@ -245,6 +245,15 @@ commands will be executed by this hook, instead of by the internal `execute_commands()` function. This hook is responsible for updating the relevant references and reporting the results back to 'receive-pack'. +RECEIVE-REPORT HOOK +------------------- +This hook is invoked by 'git-receive-pack' after all the ref updates +have been applied but before the report is sent to the client. The hook +receives the complete report in pkt-line format on stdin and its stdout +replaces the report sent to the client, which allows the hook to rewrite +the outcomes or abort the push completely. See linkgit:githooks[5] for +the full protocol description. + QUARANTINE ENVIRONMENT ---------------------- diff --git a/Documentation/githooks.adoc b/Documentation/githooks.adoc index ed045940d1..145642bf05 100644 --- a/Documentation/githooks.adoc +++ b/Documentation/githooks.adoc @@ -527,6 +527,67 @@ The exit status of the hook is ignored for any state except for the status will cause the transaction to be aborted. The hook will not be called with "aborted" state in that case. +receive-report +~~~~~~~~~~~~~~ + +This hook is invoked by linkgit:git-receive-pack[1] when it reacts to +`git push` and updates references in its repository. It executes on +the repository once after all refs have been updated and after all +accepted ref changes are applied to the repository, but before the +pkt-line encoded status report is sent back to the client. + +The hook receives the complete pkt-line encoded status report on +standard input, see linkgit:gitprotocol-pack[5] for details on the +structure. The hook's standard output entirely replaces the report +that is sent to the client. The hook must write a valid pkt-line +encoded report in the same format it received. The hook's stdout is +fully buffered by `receive-pack` before any data is sent to the client, +so the hook's exit status is known before the client receives anything. + +There are three distinct ways the hook can affect the push outcome: + +* To reject the push, modify the unpack status from `ok` to the required + error message. While `git-push` will fail, individual references may + still show success messages unless modified. + +* To reject individual ref updates while keeping `receive-pack` alive, + rewrite the corresponding `ok ` lines to + `ng [ ]` lines in the output and exit with status 0. + The client will then mark those specific refs as rejected while + treating any `ok` refs as successful. The push as a whole is + considered failed if any ref is `ng`, and `git push` will exit with + a non-zero status on the client side. + +* To abort the entire push unconditionally, exit with a non-zero + status. In this case the hook's stdout is discarded, `receive-pack` + modifies all references to be rejected with a 'receive-report hook + failed' error. + +Any output written to standard error is forwarded to the client over +the sideband channel and will appear as `remote:` lines on clients +using 'git-push(1)', regardless of the hook's exit status. Writing to +standard error alone does not affect the push outcome. + +Note that by the time this hook runs, all ref updates have already been +applied to the repository. Neither a non-zero exit nor rewriting refs +to `ng` rolls back any ref changes that were already committed +server-side. The hook can cause the client to perceive the push as +failed, but cannot undo the server-side updates. + +This means that reporting a reference as `ng` makes the client believe +the update did not happen while the server has in fact applied it. The +client leaves its remote-tracking reference at its old value, and a +later `git fetch` may reveal the very update that the push reported as +rejected. Neither Git nor the server can reconcile this; only the user, +by fetching again, will find out. + +This hook is therefore only appropriate for servers which can guarantee +that a rejected update is not observable by any reader, for example +because the committed transaction produces a candidate state that is +discarded rather than published. On a repository where a committed +reference update is immediately visible, using this hook to reject a +push will leave the pusher with a view that does not match the server. + push-to-checkout ~~~~~~~~~~~~~~~~ diff --git a/builtin/receive-pack.c b/builtin/receive-pack.c index 9ac10465ac..edfd5cf9dc 100644 --- a/builtin/receive-pack.c +++ b/builtin/receive-pack.c @@ -1008,6 +1008,41 @@ static int run_update_hook(struct command *cmd) return code; } +static int run_receive_report_hook(struct strbuf *report) +{ + struct child_process proc = CHILD_PROCESS_INIT; + struct async sideband_async; + int sideband_async_started = 0; + int saved_stderr = -1; + struct strbuf out = STRBUF_INIT; + const char *hook_path; + int ret; + + hook_path = find_hook(the_repository, "receive-report"); + if (!hook_path) + return 0; + + strvec_push(&proc.args, hook_path); + proc.trace2_hook_name = "receive-report"; + + prepare_sideband_async(&sideband_async, &saved_stderr, + &sideband_async_started); + + sigchain_push(SIGPIPE, SIG_IGN); + ret = pipe_command(&proc, report->buf, report->len, &out, + report->len, NULL, 0); + sigchain_pop(SIGPIPE); + + finish_sideband_async(&sideband_async, saved_stderr, + sideband_async_started); + + if (!ret) + strbuf_swap(&out, report); + + strbuf_release(&out); + return ret; +} + static struct command *find_command_by_refname(struct command *list, const char *refname) { @@ -2535,6 +2570,13 @@ static void update_shallow_info(struct command *commands, free(ref_status); } +static void override_cmds_error(struct command *commands, const char *err) +{ + for (struct command *cmd = commands; cmd; cmd = cmd->next) { + cmd->error_string = err; + } +} + /* * Generate the response to be sent to the client invoking 'git-receive-pack(1)'. * For v2 protocol, set `detailed_report` to true, which will also add detailed @@ -2589,6 +2631,12 @@ static void report(struct command *commands, const char *unpack_status) generate_report(&buf, commands, unpack_status, REPORT_STATUS_V0); + if (run_receive_report_hook(&buf)) { + strbuf_reset(&buf); + override_cmds_error(commands, "receive-report hook failed"); + generate_report(&buf, commands, unpack_status, false); + } + if (use_sideband) send_sideband(1, 1, buf.buf, buf.len, use_sideband); else @@ -2602,6 +2650,12 @@ static void report_v2(struct command *commands, const char *unpack_status) generate_report(&buf, commands, unpack_status, REPORT_STATUS_V2); + if (run_receive_report_hook(&buf)) { + strbuf_reset(&buf); + override_cmds_error(commands, "receive-report hook failed"); + generate_report(&buf, commands, unpack_status, true); + } + if (use_sideband) send_sideband(1, 1, buf.buf, buf.len, use_sideband); else diff --git a/t/meson.build b/t/meson.build index a25f37d2f5..7088c2c1c1 100644 --- a/t/meson.build +++ b/t/meson.build @@ -651,6 +651,7 @@ integration_tests = [ 't5409-colorize-remote-messages.sh', 't5410-receive-pack.sh', 't5411-proc-receive-hook.sh', + 't5412-receive-report-hook.sh', 't5500-fetch-pack.sh', 't5501-fetch-push-alternates.sh', 't5502-quickfetch.sh', diff --git a/t/t5412-receive-report-hook.sh b/t/t5412-receive-report-hook.sh new file mode 100755 index 0000000000..24679de37b --- /dev/null +++ b/t/t5412-receive-report-hook.sh @@ -0,0 +1,224 @@ +#!/bin/sh + +test_description='test receive-report hook' + +GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main +export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME + +. ./test-lib.sh + +. "$TEST_DIRECTORY"/t5411/common-functions.sh + +URL_PREFIX="\.\." + +test_expect_success "setup workbench" ' + git init workbench && + create_commits_in workbench A B +' + +test_expect_success "no report hook, push succeeds" ' + test_when_finished "rm -rf upstream" && + test_when_finished "git -C workbench remote remove origin" && + git init --bare upstream && + + git -C workbench remote add origin ../upstream && + git -C workbench push origin $A:refs/heads/main && + git -C workbench push origin $B:refs/heads/main >out 2>&1 && + + make_user_friendly_and_stable_output actual && + cat >expect <<-\EOF && + To ../upstream + .. -> main + EOF + test_cmp expect actual +' + +test_expect_success "passthrough does not alter report" ' + test_when_finished "rm -rf upstream" && + test_when_finished "git -C workbench remote remove origin" && + git init --bare upstream && + + test_hook -C upstream --setup receive-report <<-\EOF && + cat + EOF + + git -C workbench remote add origin ../upstream && + git -C workbench push origin $A:refs/heads/main && + git -C workbench push origin $B:refs/heads/main >out 2>&1 && + + make_user_friendly_and_stable_output actual && + cat >expect <<-\EOF && + To ../upstream + .. -> main + EOF + test_cmp expect actual +' + +test_expect_success "non-zero exit reports as hook failed" ' + test_when_finished "rm -rf upstream" && + test_when_finished "git -C workbench remote remove origin" && + + git init --bare upstream && + git -C workbench remote add origin ../upstream && + git -C workbench push origin $A:refs/heads/main && + + test_hook -C upstream --setup receive-report <<-\EOF && + exit 1 + EOF + + test_must_fail git -C workbench push origin $B:refs/heads/main >out 2>&1 && + make_user_friendly_and_stable_output actual && + cat >expect <<-\EOF && + To ../upstream + ! [remote rejected] -> main (receive-report hook failed) + EOF + test_cmp expect actual +' + +test_expect_success "hook is invoked and receives report on stdin" ' + test_when_finished "rm -rf upstream" && + test_when_finished "git -C workbench remote remove origin" && + + git init --bare upstream && + test_hook -C upstream --setup receive-report <<-EOF && + tee raw + EOF + + git -C workbench remote add origin ../upstream && + git -C workbench push origin $A:refs/heads/main && + git -C workbench push origin $B:refs/heads/main >out 2>&1 && + + make_user_friendly_and_stable_output actual && + cat >expect <<-EOF && + To ../upstream + .. -> main + EOF + test_cmp expect actual && + + test-tool pkt-line unpack actual-report && + cat >expect-report <<-EOF && + unpack ok + ok refs/heads/main + 0000 + EOF + test_cmp expect-report actual-report +' + +test_expect_success "hook can modify the report sent to client" ' + test_when_finished "rm -rf upstream" && + test_when_finished "git -C workbench remote remove origin" && + + git init --bare upstream && + git -C workbench remote add origin ../upstream && + git -C workbench push origin $A:refs/heads/main && + + test_hook -C upstream --setup receive-report <<-\EOF && + test-tool pkt-line unpack | + sed "s/^ok /ng /" | + test-tool pkt-line pack + EOF + + test_must_fail git -C workbench push origin $B:refs/heads/main >out 2>&1 && + make_user_friendly_and_stable_output actual && + cat >expect <<-\EOF && + To ../upstream + ! [remote rejected] -> main (failed) + EOF + test_cmp expect actual +' + +test_expect_success "hook can modify the unpack status" ' + test_when_finished "rm -rf upstream" && + test_when_finished "git -C workbench remote remove origin" && + + git init --bare upstream && + git -C workbench remote add origin ../upstream && + git -C workbench push origin $A:refs/heads/main && + + test_hook -C upstream --setup receive-report <<-\EOF && + test-tool pkt-line unpack | + sed "s/^unpack ok$/unpack push failed due to server error/" | + test-tool pkt-line pack + EOF + + test_must_fail git -C workbench push origin $B:refs/heads/main >out 2>&1 && + test_grep "error: remote unpack failed: push failed due to server error" out && + make_user_friendly_and_stable_output actual && + cat >expect <<-\EOF && + To ../upstream + .. -> main + EOF + test_cmp expect actual +' + +test_expect_success "hook can report a custom failure message" ' + test_when_finished "rm -rf upstream" && + test_when_finished "git -C workbench remote remove origin" && + + git init --bare upstream && + git -C workbench remote add origin ../upstream && + git -C workbench push origin $A:refs/heads/main && + + test_hook -C upstream --setup receive-report <<-\EOF && + echo "push rejected: service X is down" >&2 + test-tool pkt-line unpack | + sed "s/^ok \(.*\)/ng \1 service-x-is-down/" | + test-tool pkt-line pack | + tee raw + EOF + + test_must_fail git -C workbench push origin $B:refs/heads/main >out 2>&1 && + test_grep "push rejected: service X is down" out && + + test-tool pkt-line unpack actual-report && + cat >expect-report <<-\EOF && + unpack ok + ng refs/heads/main service-x-is-down + 0000 + EOF + test_cmp expect-report actual-report +' + +test_expect_success "hook stderr with zero exit status code" ' + test_when_finished "rm -rf upstream" && + test_when_finished "git -C workbench remote remove origin" && + + git init --bare upstream && + git -C workbench remote add origin ../upstream && + git -C workbench push origin $A:refs/heads/main && + + test_hook -C upstream --setup receive-report <<-\EOF && + echo "push rejected: service X is down" >&2 + tee raw + EOF + + git -C workbench push origin $B:refs/heads/main >out 2>&1 && + test_grep "push rejected: service X is down" out && + + test-tool pkt-line unpack actual-report && + cat >expect-report <<-\EOF && + unpack ok + ok refs/heads/main + 0000 + EOF + test_cmp expect-report actual-report +' + +test_expect_success "hook stderr is relayed to client via sideband" ' + test_when_finished "rm -rf upstream" && + test_when_finished "git -C workbench remote remove origin" && + + git init --bare upstream && + git -C workbench remote add origin ../upstream && + git -C workbench push origin $A:refs/heads/main && + + test_hook -C upstream --setup receive-report <<-\EOF && + echo "hook-stderr-message" >&2 + exit 1 + EOF + + test_must_fail git -C workbench push origin $B:refs/heads/main >out 2>&1 && + test_grep "remote: hook-stderr-message" out +' + +test_done -- 2.55.GIT