SunnyBoy-WYH commented on code in PR #2466:
URL:
https://github.com/apache/incubator-hugegraph/pull/2466#discussion_r1510233703
##########
hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/api/filter/PathFilter.java:
##########
@@ -35,24 +42,29 @@ public class PathFilter implements ContainerRequestFilter {
@Override
public void filter(ContainerRequestContext context) throws IOException {
- context.setProperty(REQUEST_TIME, System.currentTimeMillis());
+ long startTime = System.currentTimeMillis();
+
+ context.setProperty(REQUEST_TIME, startTime);
+
+ recordRequestJson(context);
+ }
- // TODO: temporarily comment it to fix loader bug, handle it later
- /*// record the request json
+ private void recordRequestJson(ContainerRequestContext context) throws
IOException {
String method = context.getMethod();
- String requestParamsJson = "";
if (method.equals(HttpMethod.POST)) {
- requestParamsJson = IOUtils.toString(context.getEntityStream(),
- Charsets.toCharset(CHARSET));
- // replace input stream because we have already read it
- InputStream in = IOUtils.toInputStream(requestParamsJson,
Charsets.toCharset(CHARSET));
- context.setEntityStream(in);
+ BufferedInputStream bufferedStream = new
BufferedInputStream(context.getEntityStream());
+
+ bufferedStream.mark(Integer.MAX_VALUE);
+
+ context.setProperty(REQUEST_PARAMS_JSON,
+ IOUtils.toString(bufferedStream,
Charsets.toCharset(CHARSET)));
Review Comment:
maybe we can cut the special length? like 512?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@hugegraph.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@hugegraph.apache.org
For additional commands, e-mail: issues-h...@hugegraph.apache.org
