opendev/gerrit
Code Issues Proposed changes

Merge "Format h/help output as plain text not JSON"

Browse Source
This commit is contained in:
Shawn O. Pearce
2012-12-02 16:22:44 -08:00
committed by Gerrit Code Review
parent 2acddbd454 0d59f736cf
commit c0b1ec4cfc
2 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gerrit-httpd/src/main/java/com/google/gerrit/httpd/restapi/ParameterParser.java
View File

@@ -14,8 +14,8 @@
package com.google.gerrit.httpd.restapi;
import static com.google.gerrit.httpd.restapi.RestApiServlet.replyBinaryResult;
import static com.google.gerrit.httpd.restapi.RestApiServlet.replyError;
import static com.google.gerrit.httpd.restapi.RestApiServlet.replyText;
import static javax.servlet.http.HttpServletResponse.SC_BAD_REQUEST;
import com.google.common.annotations.VisibleForTesting;
@@ -26,6 +26,7 @@ import com.google.common.collect.Iterables;
import com.google.common.collect.Multimap;
import com.google.common.collect.Sets;
import com.google.gerrit.extensions.restapi.BadRequestException;
import com.google.gerrit.extensions.restapi.BinaryResult;
import com.google.gerrit.server.util.Url;
import com.google.gerrit.util.cli.CmdLineParser;
import com.google.gson.JsonArray;
@@ -78,7 +79,8 @@ class ParameterParser {
msg.write('\n');
clp.printUsage(msg, null);
msg.write('\n');
replyText(req, res, msg.toString());
replyBinaryResult(req, res,
BinaryResult.create(msg.toString()).setContentType("text/plain"));
return false;
}

2
gerrit-httpd/src/main/java/com/google/gerrit/httpd/restapi/RestApiServlet.java
View File

@@ -494,7 +494,7 @@ public class RestApiServlet extends HttpServlet {
}
}
private static void replyBinaryResult(
static void replyBinaryResult(
@Nullable HttpServletRequest req,
HttpServletResponse res,
BinaryResult bin) throws IOException {