feat(relay): route git pre-receive policy callback over UDS when configured

[tlongwell-block]

Problem

The pre-receive git hook curls the relay's own /internal/git/policy callback at http://127.0.0.1:{bind_port}. In a mesh deployment (e.g. Istio on sprout), the sidecar's iptables captures that loopback port, so the relay's call to itself fails with network error reaching policy service and every push is rejected. (See companion tf-stacks PR squareup/block-coder-tf-stacks#2244 for the immediate deploy-side unblock.)

Why UDS

The relay already serves the same router over its UDS listener (main.rs, when BUZZ_UDS_PATH is set) — so /internal/git/policy is already reachable over the socket. Only the hook's curl target needs to change. A unix-socket callback dodges any in-pod L4 interception of the loopback TCP port entirely, so this can't regress when mesh config changes.

Change (~15 lines, 2 files + tests)

• transport.rs: when config.uds_path is set, point BUZZ_HOOK_URL at a placeholder host (curl ignores host:port under --unix-socket) and pass the socket path via BUZZ_HOOK_SOCKET. Extracted policy_hook_url() as a pure, unit-testable helper. When uds_path is None, behavior is byte-identical to before (TCP loopback) — local/dev/CI untouched.
• hook.rs: add --unix-socket "$BUZZ_HOOK_SOCKET" to the policy curl iff BUZZ_HOOK_SOCKET is non-empty (bash array, no word-split risk). --unix-socket is stock libcurl; the relay image ships curl.
• Unit tests for both URL branches.

Validation

• cargo build -p buzz-relay ✅, cargo clippy -p buzz-relay clean ✅
• New tests pass: policy_hook_url_uses_tcp_loopback_without_uds, policy_hook_url_uses_placeholder_host_with_uds ✅
• Embedded hook script bash -n syntax-clean; verified the --unix-socket flag is added only when the socket env is set (unset/empty → unchanged TCP path) ✅
• e2e_git.rs already exercises push→policy over TCP (no UDS in CI), so the default path stays covered.
• Note: two unrelated full-suite failures (local_echo_presence... — a config-test bogus-path fixture polluting a parallel Config::default(); owner_archive_rejects_stale... — needs a live migrated Postgres) are pre-existing env/DB issues, not touched by this diff.

Deploy follow-up

Pairs with a tf-stacks chart change that sets BUZZ_UDS_PATH=/run/buzz/relay.sock + a shared emptyDir mount at /run/buzz on the relay container. Sequence: this PR merges → image builds → flip chart to UDS → drop the #2244 excludeOutbound workaround.

Refs RESEARCH/SPROUT_GIT_POLICY_DEPLOY_MISMATCH.md.

🤖 Drafted by Eva; reviewed/tested with Max.

[tlongwell-block]

Update: review by Max found that the UDS listener served into_make_service() (no connect-info), so the /internal/git/policy route's require_localhost guard would fail closed with HTTP 403 over the socket (no ConnectInfo<SocketAddr> in extensions). Fixed in cd5da7e3e: added a UdsConnectInfo marker + into_make_service_with_connect_info::<UdsConnectInfo>() on the UDS listener, and taught require_localhost to accept either a loopback TCP SocketAddr or the UDS marker. TCP path stays fail-closed without a loopback addr. Added 4 guard unit tests (no connect-info → 403, loopback TCP → 200, non-loopback TCP → 403, UDS marker → 200).

[tlongwell-block]

🤖 Closing this due to a week of inactivity. We're getting a high volume of PRs and this just helps maintainers keep the queue manageable. It's nothing about this PR or you personally! Please reopen (or drop a comment) if it's still needed.