fetch: generate ref-prefixes when using a configured refspec
authorBrandon Williams <bmwill@google.com>
Wed, 16 May 2018 23:48:22 +0000 (16:48 -0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 17 May 2018 21:35:37 +0000 (06:35 +0900)
Teach fetch to generate ref-prefixes, to be used for server-side
filtering of the ref-advertisement, based on the configured fetch
refspec ('remote.<name>.fetch') when no user provided refspec exists.

Signed-off-by: Brandon Williams <bmwill@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/fetch.c
t/t5702-protocol-v2.sh
index 80bb143707f98be1fe7ab63a71f8599d87171dad..7cc7a52deefd5c490103e4c154e864b211c2bd29 100644 (file)
@@ -351,7 +351,15 @@ static struct ref *get_ref_map(struct transport *transport,
 
        const struct ref *remote_refs;
 
-       refspec_ref_prefixes(rs, &ref_prefixes);
+       if (rs->nr)
+               refspec_ref_prefixes(rs, &ref_prefixes);
+       else if (transport->remote && transport->remote->fetch.nr)
+               refspec_ref_prefixes(&transport->remote->fetch, &ref_prefixes);
+
+       if (ref_prefixes.argc &&
+           (tags == TAGS_SET || (tags == TAGS_DEFAULT && !rs->nr))) {
+               argv_array_push(&ref_prefixes, "refs/tags/");
+       }
 
        remote_refs = transport_get_remote_refs(transport, &ref_prefixes);
 
index 56f7c3c326cc045da5d443ff9ad1cb5653fcf21f..b6c72ab51ed2f4ab3012252b70c6b03fafbcaf65 100755 (executable)
@@ -201,6 +201,20 @@ test_expect_success 'ref advertisment is filtered during fetch using protocol v2
        ! grep "refs/tags/three" log
 '
 
+test_expect_success 'default refspec is used to filter ref when fetchcing' '
+       test_when_finished "rm -f log" &&
+
+       GIT_TRACE_PACKET="$(pwd)/log" git -C file_child -c protocol.version=2 \
+               fetch origin &&
+
+       git -C file_child log -1 --format=%s three >actual &&
+       git -C file_parent log -1 --format=%s three >expect &&
+       test_cmp expect actual &&
+
+       grep "ref-prefix refs/heads/" log &&
+       grep "ref-prefix refs/tags/" log
+'
+
 # Test protocol v2 with 'http://' transport
 #
 . "$TEST_DIRECTORY"/lib-httpd.sh