Mailing-List: contact dev-help@chemistry.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@chemistry.apache.org
Date: Mon, 22 Sep 2014 15:57:34 +0000 (UTC)
From: "Ron Gavlin (JIRA)" <jira@apache.org>
To: dev@chemistry.apache.org
Message-ID: <JIRA.12742920.1411189727000.94289.1411401454932@Atlassian.JIRA>
In-Reply-To: <JIRA.12742920.1411189727000@Atlassian.JIRA>
References: <JIRA.12742920.1411189727000@Atlassian.JIRA>
 <JIRA.12742920.1411189727435@arcas>
Subject: [jira] [Commented] (CMIS-844) POSTHttpServletRequestWrapper should
 accommodate prior consumption of request body by a filter
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable


    [ https://issues.apache.org/jira/browse/CMIS-844?page=3Dcom.atlassian.j=
ira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=3D141433=
36#comment-14143336 ]=20

Ron Gavlin commented on CMIS-844:
---------------------------------

You are correct that saving the original stream in the filter is an option =
so that OpenCMIS can decode independently of the filter. However, the doubl=
e-decoding does introduce overhead and requires changes to existing code th=
at formerly worked with the 0.7.0 release. Would you consider defining a co=
nfigurable property to optionally enable the functionality provided by the =
patch?

> POSTHttpServletRequestWrapper should accommodate prior consumption of req=
uest body by a filter
> -------------------------------------------------------------------------=
---------------------
>
>                 Key: CMIS-844
>                 URL: https://issues.apache.org/jira/browse/CMIS-844
>             Project: Chemistry
>          Issue Type: Bug
>          Components: opencmis-server
>    Affects Versions: OpenCMIS 0.12.0
>            Reporter: Ron Gavlin
>            Assignee: Florian M=C3=BCller
>
> POSTHttpServletRequestWrapper should accommodate prior consumption of the=
 request body by a filter.
> The 0.7.0 release provided this accommodation. This appears to have been =
broken by the major post-0.7.0 refactoring.
> The following POSTHttpServletRequestWrapper constructor modification fixe=
s the problem:
> {code}
> ...
>         if (isMultipart) {
> ...
>         } else {
>             // form data processing
>             StringBuilder sb =3D new StringBuilder();
>             InputStreamReader sr =3D new InputStreamReader(request.getInp=
utStream(), IOUtils.UTF8);
>             char[] buffer =3D new char[4096];
>             int c =3D 0;
>             while ((c =3D sr.read(buffer)) > -1) {
>                 sb.append(buffer, 0, c);
>             }
>             if (sb.length() < 3) {
>                 @SuppressWarnings("unchecked")
>                 Map<String, String[]> parameterMap =3D request.getParamet=
erMap();
>                 getParameterMap().putAll(parameterMap);
>             } else {
>                 parseFormData(sb.toString());
>             }
>         }
> ...
> {code}


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)