You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by si...@apache.org on 2012/04/26 14:17:58 UTC
svn commit: r1330803 - in /lucene/dev/trunk/solr/solrj/src:
java/org/apache/solr/client/solrj/impl/HttpSolrServer.java
test/org/apache/solr/client/solrj/SolrExampleTests.java
Author: siren
Date: Thu Apr 26 12:17:58 2012
New Revision: 1330803
URL: http://svn.apache.org/viewvc?rev=1330803&view=rev
Log:
SOLR-2020: fix another bug with param encoding with multipart request
Modified:
lucene/dev/trunk/solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpSolrServer.java
lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/client/solrj/SolrExampleTests.java
Modified: lucene/dev/trunk/solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpSolrServer.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpSolrServer.java?rev=1330803&r1=1330802&r2=1330803&view=diff
==============================================================================
--- lucene/dev/trunk/solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpSolrServer.java (original)
+++ lucene/dev/trunk/solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpSolrServer.java Thu Apr 26 12:17:58 2012
@@ -294,7 +294,7 @@ public class HttpSolrServer extends Solr
}
if (parts.size() > 0) {
- MultipartEntity entity = new MultipartEntity(HttpMultipartMode.BROWSER_COMPATIBLE);
+ MultipartEntity entity = new MultipartEntity(HttpMultipartMode.STRICT);
for(FormBodyPart p: parts) {
entity.addPart(p);
}
Modified: lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/client/solrj/SolrExampleTests.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/client/solrj/SolrExampleTests.java?rev=1330803&r1=1330802&r2=1330803&view=diff
==============================================================================
--- lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/client/solrj/SolrExampleTests.java (original)
+++ lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/client/solrj/SolrExampleTests.java Thu Apr 26 12:17:58 2012
@@ -613,6 +613,17 @@ abstract public class SolrExampleTests e
assertNotNull("Couldn't upload xml files", result);
rsp = server.query( new SolrQuery( "*:*") );
Assert.assertEquals( 5 , rsp.getResults().getNumFound() );
+
+ //params encoding, multipart
+ up = new ContentStreamUpdateRequest("/debug/dump");
+ up.addFile(getFile("solrj/docs1.xml")); // 2
+ up.addFile(getFile("solrj/docs2.xml")); // 3
+ up.setParam("a", "\u1234");
+ result = server.request(up);
+ System.out.println(result);
+ Assert.assertEquals("\u1234",
+ ((NamedList)((NamedList) result.get("responseHeader")).get("params")).get("a"));
+
}