Ryan <ryanphilips19@xxxxxxxxxxxxxx> writes:
> Hi,
> Thank you for your response.
>
> 1) git config user.name 'Matthieu M. Moy' Works
> 2) git config user.name 'Matthieu M.' does not
>
> Name cannot end in a '.' <dot>. in git config user.name
That is correct, but I think we are a bit too strict in sanitizing names.
The code is ident.c::copy() whose intent was to sanitize useless runs of
commas we slurp from GECOS field, but the sanitization is done also for
anything specified by the end user.
This patch loosens it, but I think it goes too far, in that it does not
stop you from feeding total nonsense with GIT_AUTHOR_NAME environment and
user.name configuration.
ident.c | 5 ++---
t/t1300-repo-config.sh | 12 ++++++++++++
2 files changed, 14 insertions(+), 3 deletions(-)
diff --git a/ident.c b/ident.c
index 99f1c85..07ec81d 100644
--- a/ident.c
+++ b/ident.c
@@ -212,8 +212,7 @@ const char *fmt_ident(const char *name, const char *email,
pw = getpwuid(getuid());
if (!pw)
die("You don't exist. Go away!");
- strlcpy(git_default_name, pw->pw_name,
- sizeof(git_default_name));
+ copy(git_default_name, sizeof(git_default_name), 0, pw->pw_name);
name = git_default_name;
}
@@ -221,7 +220,7 @@ const char *fmt_ident(const char *name, const char *email,
if (!name_addr_only && date_str)
parse_date(date_str, date, sizeof(date));
- i = copy(buffer, sizeof(buffer), 0, name);
+ i = add_raw(buffer, sizeof(buffer), 0, name);
i = add_raw(buffer, sizeof(buffer), i, " <");
i = copy(buffer, sizeof(buffer), i, email);
if (!name_addr_only) {
diff --git a/t/t1300-repo-config.sh b/t/t1300-repo-config.sh
index 43ea283..c1a913e 100755
--- a/t/t1300-repo-config.sh
+++ b/t/t1300-repo-config.sh
@@ -758,4 +758,16 @@ test_expect_success 'check split_cmdline return' "
test_must_fail git merge master
"
+test_expect_success 'name ending in dot' '
+ foobar="Foo Bar."
+ git config user.name "$foobar" &&
+ git config user.email "foo.bar@xxxxxxxxxxx" &&
+ test "$(git config user.name)" = "$foobar" &&
+ (
+ unset GIT_AUTHOR_NAME
+ ident=$(git var GIT_AUTHOR_IDENT | sed -e "s/ *<.*//") &&
+ test "$ident" = "$foobar"
+ )
+'
+
test_done
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html