Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] send-email: add extra safetly in address sanitazion

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:52:57

Thomas Rast [off-list ref] writes:
Am I the only one who stared at this for ten seconds, only to then
realize that there is no sanitizing whatsoever going on here?
quoted
  'Foo Bar [off-list ref]>' -> 'Foo Bar [off-list ref]'
  '"Foo Bar" [off-list ref]>' -> '"Foo Bar" [off-list ref]'
  '[off-list ref]>' -> 'foo@bar.com'
All of these are the same underlying issue.  Does your patch fix any
other malformed addresses, or just this particular type?
Just this particular type, as long as the code handles it correctly, would
be better than nothing.

On the recieving end in mailinfo, I think we also support

	gitster@pobox.com (Junio C Hamano)

although it does not seem to be used by many people these days (the only
one I can remember seeing on this list was merlyn), so we may want to be a
bit more consistent between sending and receiving end, though.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help