chemistry-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Florent Guillaume>
Subject POSTHttpServletRequestWrapper parameters
Date Mon, 22 Sep 2014 13:36:32 GMT

When POSTHttpServletRequestWrapper is invoked, it reads all the
parameters from a POST body into its own parameter map
(QueryStringHttpServletRequestWrapper), by consuming and parsing the
input stream.

However in our implementation the input stream was already read and
parsed by a previous filter that needed to consult the request
parameters through a call to request.getParameter().

I suggest that POSTHttpServletRequestWrapper should allow for a
previous filter having already parsed the request parameters and
consumed the body, by at least delegating to super for the initial
acquisition of its parameter map, in

What do you think? I can provide a patch if needed.

(As has been pointed out to me, in theory the previous filter
could/should have wrapped the whole request to cache and return the
body, but here we're dealing with multipart/form-data and the body has
been turned into a parameter map and is still available in another


Florent Guillaume, Director of R&D, Nuxeo
Open Source Content Management Platform for Business Apps

View raw message