Merge branch 'js/reflog-anonymize-for-clone-and-fetch'
The reflog entries for "git clone" and "git fetch" did not anonymize the URL they operated on. * js/reflog-anonymize-for-clone-and-fetch: clone/fetch: anonymize URLs in the reflog
This commit is contained in:
@ -945,7 +945,7 @@ int cmd_clone(int argc, const char **argv, const char *prefix)
|
||||
{
|
||||
int is_bundle = 0, is_local;
|
||||
const char *repo_name, *repo, *work_tree, *git_dir;
|
||||
char *path, *dir;
|
||||
char *path, *dir, *display_repo = NULL;
|
||||
int dest_exists;
|
||||
const struct ref *refs, *remote_head;
|
||||
const struct ref *remote_head_points_at;
|
||||
@ -1000,10 +1000,11 @@ int cmd_clone(int argc, const char **argv, const char *prefix)
|
||||
path = get_repo_path(repo_name, &is_bundle);
|
||||
if (path)
|
||||
repo = absolute_pathdup(repo_name);
|
||||
else if (!strchr(repo_name, ':'))
|
||||
die(_("repository '%s' does not exist"), repo_name);
|
||||
else
|
||||
else if (strchr(repo_name, ':')) {
|
||||
repo = repo_name;
|
||||
display_repo = transport_anonymize_url(repo);
|
||||
} else
|
||||
die(_("repository '%s' does not exist"), repo_name);
|
||||
|
||||
/* no need to be strict, transport_set_option() will validate it again */
|
||||
if (option_depth && atoi(option_depth) < 1)
|
||||
@ -1020,7 +1021,9 @@ int cmd_clone(int argc, const char **argv, const char *prefix)
|
||||
die(_("destination path '%s' already exists and is not "
|
||||
"an empty directory."), dir);
|
||||
|
||||
strbuf_addf(&reflog_msg, "clone: from %s", repo);
|
||||
strbuf_addf(&reflog_msg, "clone: from %s",
|
||||
display_repo ? display_repo : repo);
|
||||
free(display_repo);
|
||||
|
||||
if (option_bare)
|
||||
work_tree = NULL;
|
||||
|
Reference in New Issue
Block a user