use a default ref-prefix when fetching with git protocol v2 (#1404)

Github's SSH git protocol v2 server returns a useless reference list which only contains "HEAD" unless we explicitly ask for the "refs/" prefix. Fixes issue #1389
jelmer · Oct 30, 2024 · e2ccec6 · e2ccec6
2 parents d13d454 + 2447376
commit e2ccec6
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/dulwich/client.py b/dulwich/client.py
@@ -832,7 +832,7 @@ def clone(
         branch=None,
         progress=None,
         depth=None,
-        ref_prefix=[],
+        ref_prefix=[b"HEAD", b"refs/"],
         filter_spec=None,
         protocol_version: Optional[int] = None,
     ) -> Repo:
@@ -928,7 +928,7 @@ def fetch(
         ] = None,
         progress: Optional[Callable[[bytes], None]] = None,
         depth: Optional[int] = None,
-        ref_prefix: Optional[List[bytes]] = [],
+        ref_prefix: Optional[List[bytes]] = [b"HEAD", b"refs/"],
         filter_spec: Optional[bytes] = None,
         protocol_version: Optional[int] = None,
     ) -> FetchPackResult:
@@ -1009,7 +1009,7 @@ def fetch_pack(
         *,
         progress: Optional[Callable[[bytes], None]] = None,
         depth: Optional[int] = None,
-        ref_prefix=[],
+        ref_prefix=[b"HEAD", b"refs/"],
         filter_spec=None,
         protocol_version: Optional[int] = None,
     ):
@@ -1311,7 +1311,7 @@ def fetch_pack(
         pack_data,
         progress=None,
         depth=None,
-        ref_prefix=[],
+        ref_prefix=[b"HEAD", b"refs/"],
         filter_spec=None,
         protocol_version: Optional[int] = None,
     ):
@@ -1791,7 +1791,7 @@ def fetch(
         determine_wants=None,
         progress=None,
         depth=None,
-        ref_prefix=[],
+        ref_prefix=[b"HEAD", b"refs/"],
         filter_spec=None,
         **kwargs,
     ):
@@ -1836,7 +1836,7 @@ def fetch_pack(
         pack_data,
         progress=None,
         depth=None,
-        ref_prefix: Optional[List[bytes]] = [],
+        ref_prefix: Optional[List[bytes]] = [b"HEAD", b"refs/"],
         filter_spec: Optional[bytes] = None,
         protocol_version: Optional[int] = None,
     ) -> FetchPackResult:
@@ -2574,7 +2574,7 @@ def fetch_pack(
         pack_data,
         progress=None,
         depth=None,
-        ref_prefix=[],
+        ref_prefix=[b"HEAD", b"refs/"],
         filter_spec=None,
         protocol_version: Optional[int] = None,
     ):