----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24449/#review50033 -----------------------------------------------------------
/trunk/java/common/src/test/java/org/apache/shindig/protocol/JsonRpcServletTest.java <https://reviews.apache.org/r/24449/#comment87538> Instead of changing the behavior of the existing tests, can you add an additional test for using the default encoding? I want to ensure the old behavior stands when DiskFileItem.getCharSet() != null - Stanton Sievers On Aug. 7, 2014, 1:48 p.m., Andreas Kohn wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24449/ > ----------------------------------------------------------- > > (Updated Aug. 7, 2014, 1:48 p.m.) > > > Review request for shindig. > > > Bugs: SHINDIG-1981 > https://issues.apache.org/jira/browse/SHINDIG-1981 > > > Repository: shindig > > > Description > ------- > > When using RPC requests with multipart/form-data encoding (due to file > uploads) the request content is currently assumed to either have an explicit > Content-Type with a 'charset' parameter set, or be encoded in ISO-8859-1. > > With HTML 5 this breaks: > 1. non-file items "must not" have any content-type header, and > 2. the encoding should come from the encoding of the form (accept-charset), > and > 3. if that is not known the default is _UTF-8_. > > The attached patch changes the behavior of JsonRpcServlet and the > CommonsFormDataItem to handle this case: > 1. JsonRpcServlet provides the request encoding when getting the content of a > non-file item as string, and > 2. CommonsFormDataItem uses that encoding unless there is an explicit > encoding known for that specific item > > > Diffs > ----- > > > /trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java > 1616468 > > /trunk/java/common/src/main/java/org/apache/shindig/protocol/multipart/CommonsFormDataItem.java > 1616468 > > /trunk/java/common/src/main/java/org/apache/shindig/protocol/multipart/FormDataItem.java > 1616468 > > /trunk/java/common/src/test/java/org/apache/shindig/protocol/JsonRpcServletTest.java > 1616468 > > Diff: https://reviews.apache.org/r/24449/diff/ > > > Testing > ------- > > Used in our application with uses RPC with multipart/form-data encoding and > non-ASCII characters in the 'request' content. > > Tested with Firefox nightly on Linux, other browser tests are still ongoing. > > > Thanks, > > Andreas Kohn > >