Merge pull request #200 from lanwen/lost_body_write

fix for unused json map when method with body, but body is null

src/main/java/org/kohsuke/github/Requester.java

@@ -53,6 +53,7 @@
 import java.util.regex.Pattern;
 import java.util.zip.GZIPInputStream;
 
+import static java.util.Arrays.asList;
 import static org.kohsuke.github.GitHub.*;
 
 /**
@@ -61,6 +62,8 @@
  * @author Kohsuke Kawaguchi
  */
 class Requester {
+    private static final List<String> METHODS_WITHOUT_BODY = asList("GET", "DELETE");
+
     private final GitHub root;
     private final List<Entry> args = new ArrayList<Entry>();
     private final Map<String,String> headers = new LinkedHashMap<String, String>();
@@ -208,7 +211,7 @@ public <T> T to(String tailApiUrl, Class<T> type, String method) throws IOExcept
 
     private <T> T _to(String tailApiUrl, Class<T> type, T instance) throws IOException {
         while (true) {// loop while API rate limit is hit
-            if (method.equals("GET") && !args.isEmpty()) {
+            if (METHODS_WITHOUT_BODY.contains(method) && !args.isEmpty()) {
                 StringBuilder qs=new StringBuilder();
                 for (Entry arg : args) {
                     qs.append(qs.length()==0 ? '?' : '&');
@@ -296,6 +299,7 @@ private void buildRequest() throws IOException {
                 for (Entry e : args) {
                     json.put(e.key, e.value);
                 }
+                MAPPER.writeValue(uc.getOutputStream(), json);
             } else {
                 try {
                     byte[] bytes = new byte[32768];
@@ -311,9 +315,7 @@ private void buildRequest() throws IOException {
     }
 
     private boolean isMethodWithBody() {
-        if (method.equals("GET"))       return false;
-        if (method.equals("DELETE"))    return false;
-        return true;
+        return !METHODS_WITHOUT_BODY.contains(method);
     }
 
     /**