aboutsummaryrefslogtreecommitdiffstats
path: root/builtin/upload-pack.c
diff options
context:
space:
mode:
authorJunio C Hamano <gitster@pobox.com>2024-12-10 10:04:55 +0900
committerJunio C Hamano <gitster@pobox.com>2024-12-10 10:04:55 +0900
commit35f40385e441d5aa885f5aa813539d5ed9dc2d26 (patch)
tree07dbb374eaef5dbdff5a0ad6369232f975acd570 /builtin/upload-pack.c
parent9cd1e2e1a0350a3b6cbc4bcd268f0cbbe8c89687 (diff)
parent0ffb5a6bf1b0fd9ce0c0b1fd9ce9fd30b89a2563 (diff)
downloadgit-35f40385e441d5aa885f5aa813539d5ed9dc2d26.tar.gz
Merge branch 'bc/allow-upload-pack-from-other-people'
Loosen overly strict ownership check introduced in the recent past, to keep the promise "cloning a suspicious repository is a safe first step to inspect it". * bc/allow-upload-pack-from-other-people: Allow cloning from repositories owned by another user
Diffstat (limited to 'builtin/upload-pack.c')
-rw-r--r--builtin/upload-pack.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/builtin/upload-pack.c b/builtin/upload-pack.c
index 3b6c83fbce..dd63d6eadf 100644
--- a/builtin/upload-pack.c
+++ b/builtin/upload-pack.c
@@ -39,6 +39,7 @@ int cmd_upload_pack(int argc,
N_("interrupt transfer after <n> seconds of inactivity")),
OPT_END()
};
+ unsigned enter_repo_flags = ENTER_REPO_ANY_OWNER_OK;
packet_trace_identity("upload-pack");
disable_replace_refs();
@@ -54,7 +55,9 @@ int cmd_upload_pack(int argc,
dir = argv[0];
- if (!enter_repo(dir, strict))
+ if (strict)
+ enter_repo_flags |= ENTER_REPO_STRICT;
+ if (!enter_repo(dir, enter_repo_flags))
die("'%s' does not appear to be a git repository", dir);
switch (determine_protocol_version_server()) {