https://issues.apache.org/bugzilla/show_bug.cgi?id=53359

--- Comment #8 from Christopher Schultz <ch...@christopherschultz.net> ---
(In reply to comment #6)
> That is the only place this gets processed but you'll need to do the same
> with the split("=") two lines as well for completeness.

I'm happy to do that, but didn't want the patch to grow bigger than the
original requirements. It makes perfect sense to me to trim whitespace around
the '='.

> A test case would be nice too as would an update to the docs on hoe white
> space will be handled :)

Of course. I mainly wanted to verify that this was the only place a functional
change needed to be made.

(In reply to comment #7)
> I think you should also beware of empty strings (produced by ',,' or by an
> extra ',' at the end of values) and ignore them.

Noted.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to