Merge pull request AsyncHttpClient#36 from BluePyth/emptymaps

Handles params & queryParams only if maps are not empty

Author: jfarcand

src/main/java/com/ning/http/client/RequestBuilderBase.java

@@ -134,7 +134,7 @@ private String toUrl(boolean encode) {
                 throw new IllegalArgumentException("Illegal URL: " + url, e);
             }
 
-            if (queryParams != null) {
+            if (queryParams != null && !queryParams.isEmpty()) {
 
                 StringBuilder builder = new StringBuilder();
                 if (!url.substring(8).contains("/")) { // no other "/" than http[s]:// -> http://localhost:1234

src/main/java/com/ning/http/client/providers/netty/NettyAsyncHttpProvider.java

@@ -712,7 +712,7 @@ private static HttpRequest construct(AsyncHttpClientConfig config,
                     int length = lengthWrapper[0];
                     nettyRequest.setHeader(HttpHeaders.Names.CONTENT_LENGTH, String.valueOf(length));
                     nettyRequest.setContent(ChannelBuffers.wrappedBuffer(bytes, 0, length));
-                } else if (request.getParams() != null) {
+                } else if (request.getParams() != null && !request.getParams().isEmpty()) {
                     StringBuilder sb = new StringBuilder();
                     for (final Entry<String, List<String>> paramEntry : request.getParams()) {
                         final String key = paramEntry.getKey();