[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#373218: qa.debian.org: Patch



Raphael Hertzog <hertzog@debian.org> writes:
> On Thu, 28 Jun 2007, Russ Allbery wrote:

>> You can't drop that tr///.  It's what takes care of all the special
>> characters that otherwise need escaping.

> Since he extracts the email part, it's less of a problem.

Well, he extracts everything between <>, but I believe we still lose if,
for instance, there's a # in the e-mail address (which is an entirely
valid RFC 2822 character).  I'm a little worried about +, which is a very
common character and sometimes has special interpretations in URLs.

-- 
Russ Allbery (rra@debian.org)               <http://www.eyrie.org/~eagle/>



Reply to: