[ https://issues.apache.org/jira/browse/SCB-314?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16355149#comment-16355149 ]
ASF GitHub Bot commented on SCB-314: ------------------------------------ jeho0815 closed pull request #541: [SCB-314] Support custom rest transport server compress and max header … URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/541 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/demo/demo-springmvc/springmvc-client/src/main/java/org/apache/servicecomb/demo/springmvc/client/SpringmvcClient.java b/demo/demo-springmvc/springmvc-client/src/main/java/org/apache/servicecomb/demo/springmvc/client/SpringmvcClient.java index 71f885f32..ab1b35ce8 100644 --- a/demo/demo-springmvc/springmvc-client/src/main/java/org/apache/servicecomb/demo/springmvc/client/SpringmvcClient.java +++ b/demo/demo-springmvc/springmvc-client/src/main/java/org/apache/servicecomb/demo/springmvc/client/SpringmvcClient.java @@ -97,8 +97,11 @@ public static void run() { .check(true, metric.getInstanceMetric().getSystemMetric().getHeapUsed() != 0); TestMgr.check(true, metric.getProducerMetrics().size() > 0); TestMgr.check(true, - metric.getProducerMetrics().get("springmvc.codeFirst.saySomething").getProducerCall() - .getTotalValue(MetricsDimension.DIMENSION_STATUS, MetricsDimension.DIMENSION_STATUS_ALL).getValue() > 0); + metric.getProducerMetrics() + .get("springmvc.codeFirst.saySomething") + .getProducerCall() + .getTotalValue(MetricsDimension.DIMENSION_STATUS, MetricsDimension.DIMENSION_STATUS_ALL) + .getValue() > 0); } catch (Exception e) { TestMgr.check("true", "false"); } diff --git a/demo/demo-springmvc/springmvc-server/src/main/java/org/apache/servicecomb/demo/springmvc/server/CodeFirstSpringmvc.java b/demo/demo-springmvc/springmvc-server/src/main/java/org/apache/servicecomb/demo/springmvc/server/CodeFirstSpringmvc.java index 103e37e76..e512db9e3 100644 --- a/demo/demo-springmvc/springmvc-server/src/main/java/org/apache/servicecomb/demo/springmvc/server/CodeFirstSpringmvc.java +++ b/demo/demo-springmvc/springmvc-server/src/main/java/org/apache/servicecomb/demo/springmvc/server/CodeFirstSpringmvc.java @@ -220,6 +220,18 @@ public String sayHi(@PathVariable(name = "name") String name) { return name + " sayhi"; } + @RequestMapping(path = "/sayhi/compressed/{name}/v2", method = RequestMethod.GET) + public String sayHiForCompressed(@PathVariable(name = "name") String name) { + String bigText = + "This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text," + + "This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text," + + "This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text," + + "This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text," + + "This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text," + + "This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text,This is a big text!"; + return name + " sayhi compressed:" + bigText; + } + @RequestMapping(path = "/sayhi/{name}/v2", method = RequestMethod.PUT) public String sayHi2(@PathVariable(name = "name") String name) { return name + " sayhi 2"; diff --git a/demo/demo-springmvc/springmvc-server/src/main/resources/microservice.yaml b/demo/demo-springmvc/springmvc-server/src/main/resources/microservice.yaml index d7a69d826..651631078 100644 --- a/demo/demo-springmvc/springmvc-server/src/main/resources/microservice.yaml +++ b/demo/demo-springmvc/springmvc-server/src/main/resources/microservice.yaml @@ -20,7 +20,7 @@ servicecomb: APPLICATION_ID: springmvctest service_description: name: springmvc - version: 0.0.3 + version: 0.0.4 paths: - path: /test1/testpath property: @@ -46,6 +46,8 @@ cse: autodiscovery: true rest: address: 0.0.0.0:8080?sslEnabled=true + server: + compression: true highway: address: 0.0.0.0:7070?sslEnabled=true handler: diff --git a/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/RestServerVerticle.java b/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/RestServerVerticle.java index d011e8a83..73eab866a 100644 --- a/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/RestServerVerticle.java +++ b/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/RestServerVerticle.java @@ -124,6 +124,8 @@ private HttpServerOptions createDefaultHttpServerOptions() { HttpServerOptions serverOptions = new HttpServerOptions(); serverOptions.setUsePooledBuffers(true); serverOptions.setIdleTimeout(TransportConfig.getConnectionIdleTimeoutInSeconds()); + serverOptions.setCompressionSupported(TransportConfig.getCompressed()); + serverOptions.setMaxHeaderSize(TransportConfig.getMaxHeaderSize()); if (endpointObject.isSslEnabled()) { SSLOptionFactory factory = diff --git a/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/TransportConfig.java b/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/TransportConfig.java index e723ef093..75bcc98af 100644 --- a/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/TransportConfig.java +++ b/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/TransportConfig.java @@ -21,6 +21,8 @@ import com.netflix.config.DynamicPropertyFactory; import com.netflix.config.DynamicStringProperty; +import io.vertx.core.http.HttpServerOptions; + public final class TransportConfig { private TransportConfig() { } @@ -42,4 +44,16 @@ public static int getConnectionIdleTimeoutInSeconds() { .getIntProperty("cse.rest.server.connection.idleTimeoutInSeconds", 60) .get(); } + + public static boolean getCompressed() { + return DynamicPropertyFactory.getInstance() + .getBooleanProperty("cse.rest.server.compression", false) + .get(); + } + + public static int getMaxHeaderSize() { + return DynamicPropertyFactory.getInstance() + .getIntProperty("cse.rest.server.maxHeaderSize", HttpServerOptions.DEFAULT_MAX_HEADER_SIZE) + .get(); + } } diff --git a/transports/transport-rest/transport-rest-vertx/src/test/java/org/apache/servicecomb/transport/rest/vertx/TestTransportConfig.java b/transports/transport-rest/transport-rest-vertx/src/test/java/org/apache/servicecomb/transport/rest/vertx/TestTransportConfig.java index 514fb1398..e63dcd5b8 100644 --- a/transports/transport-rest/transport-rest-vertx/src/test/java/org/apache/servicecomb/transport/rest/vertx/TestTransportConfig.java +++ b/transports/transport-rest/transport-rest-vertx/src/test/java/org/apache/servicecomb/transport/rest/vertx/TestTransportConfig.java @@ -63,4 +63,14 @@ public void testGetThreadCountNormal() { Assert.assertEquals(10, TransportConfig.getThreadCount()); config.clearProperty("cse.rest.server.thread-count"); } + + @Test + public void testGetCompressedAndHeaderSize() { + config.addProperty("cse.rest.server.compression", true); + Assert.assertEquals(true, TransportConfig.getCompressed()); + config.addProperty("cse.rest.server.maxHeaderSize", 2048); + Assert.assertEquals(2048, TransportConfig.getMaxHeaderSize()); + config.clearProperty("cse.rest.server.compression"); + config.clearProperty("cse.rest.server.maxHeaderSize"); + } } ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > As a SDK user, i want to set the vertx compressd mode on rest transport so > that I can use ServiceComb to compresse server date > ------------------------------------------------------------------------------------------------------------------------------ > > Key: SCB-314 > URL: https://issues.apache.org/jira/browse/SCB-314 > Project: Apache ServiceComb > Issue Type: New Feature > Components: Java-Chassis > Reporter: jeho0815 > Assignee: jeho0815 > Priority: Major > Labels: consumer > > vertx can set server compressed mode, some one want to use this feature and > set the max header size -- This message was sent by Atlassian JIRA (v7.6.3#76005)