> I just think if the servlet spec says it is immutable we should keep it
> immutable.
> Then the bad code is after that. So the code that does the call to
> getParameterMap
> that should make first a clone..
Well, *our* code actually removes parameters. If we fix that, I'm fine
with making/ keeping it immutable. The only extra thing we do then is
translate single array strings to plain strings.
but that is in the RequestParameter map. Not directly the HttpRequest map
(at least it shouldn't be)
johan