{"thread":{"id":"12904","subject":"[PATCH 1/1] Fix recipient santitization","startedAt":"2008-03-28T14:09:04Z","lastAt":"2008-05-21T19:38:06Z","messageCount":2,"participants":["Horst H. von Brand","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":1},"messages":[{"id":"73272","messageId":"1206713344-11244-1-git-send-email-vonbrand@inf.utfsm.cl","threadId":"12904","inReplyTo":null,"subject":"[PATCH 1/1] Fix recipient santitization","fromName":"Horst H. von Brand","fromEmail":"vonbrand@inf.utfsm.cl","sentAt":"2008-03-28T14:09:04Z","receivedAt":"2008-03-28T14:09:04Z","isPatch":true,"sender":{"key":"vonbrand@inf.utfsm.cl","avatar":"https://avatars.githubusercontent.com/u/211384?v=4"},"body":"Need to quote all special characters, not just the first one\n\nSigned-off-by: Horst H. von Brand <vonbrand@inf.utfsm.cl>\n---\n git-send-email.perl |    2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex f62f119..3bdd225 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -635,7 +635,7 @@ sub sanitize_address\n \n \t# double quotes are needed if specials or CTLs are included\n \telsif ($recipient_name =~ /[][()<>@,;:\\\\\".\\000-\\037\\177]/) {\n-\t\t$recipient_name =~ s/([\"\\\\\\r])/\\\\$1/;\n+\t\t$recipient_name =~ s/([\"\\\\\\r])/\\\\$1/g;\n \t\t$recipient_name = \"\\\"$recipient_name\\\"\";\n \t}\n \n-- \n1.5.5.rc2\n"},{"id":"77412","messageId":"7vzlqj1m8x.fsf@gitster.siamese.dyndns.org","threadId":"12904","inReplyTo":"1206713344-11244-1-git-send-email-vonbrand@inf.utfsm.cl","subject":"Re: [PATCH 1/1] Fix recipient santitization","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-05-21T19:38:06Z","receivedAt":"2008-05-21T19:38:06Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Last night I was going through old mail-logs and found this, which I think\nstill is needed.  Does anybody see anything wrong with the patch (aside\nfrom missing full-stop at the end of the sentence)?\n\n\"Horst H. von Brand\" <vonbrand@inf.utfsm.cl> writes:\n\n> Need to quote all special characters, not just the first one\n>\n> Signed-off-by: Horst H. von Brand <vonbrand@inf.utfsm.cl>\n> ---\n>  git-send-email.perl |    2 +-\n>  1 files changed, 1 insertions(+), 1 deletions(-)\n>\n> diff --git a/git-send-email.perl b/git-send-email.perl\n> index f62f119..3bdd225 100755\n> --- a/git-send-email.perl\n> +++ b/git-send-email.perl\n> @@ -635,7 +635,7 @@ sub sanitize_address\n>  \n>  \t# double quotes are needed if specials or CTLs are included\n>  \telsif ($recipient_name =~ /[][()<>@,;:\\\\\".\\000-\\037\\177]/) {\n> -\t\t$recipient_name =~ s/([\"\\\\\\r])/\\\\$1/;\n> +\t\t$recipient_name =~ s/([\"\\\\\\r])/\\\\$1/g;\n>  \t\t$recipient_name = \"\\\"$recipient_name\\\"\";\n>  \t}\n>  \n> -- \n> 1.5.5.rc2\n"}]}