[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-12-04 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16709659#comment-16709659
 ] 

ASF GitHub Bot commented on SCB-1051:
-

wujimin closed pull request #1015: [SCB-1051]when interface set 
produces=text/plain;charset=utf-8. and c…
URL: https://github.com/apache/servicecomb-java-chassis/pull/1015
 
 
   

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/common/common-rest/src/main/java/org/apache/servicecomb/common/rest/definition/RestOperationMeta.java
 
b/common/common-rest/src/main/java/org/apache/servicecomb/common/rest/definition/RestOperationMeta.java
index ee7c40137..36979ea92 100644
--- 
a/common/common-rest/src/main/java/org/apache/servicecomb/common/rest/definition/RestOperationMeta.java
+++ 
b/common/common-rest/src/main/java/org/apache/servicecomb/common/rest/definition/RestOperationMeta.java
@@ -173,6 +173,9 @@ protected void createProduceProcessors() {
   }
 } else {
   for (String produce : produces) {
+if (produce.contains(";")) {
+  produce = produce.substring(0, produce.indexOf(";"));
+}
 ProduceProcessor processor = 
ProduceProcessorManager.INSTANCE.findValue(produce);
 if (processor == null) {
   LOGGER.error("produce {} is not supported", produce);
diff --git 
a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
 
b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
index 34621d6e2..6103df9ed 100644
--- 
a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
+++ 
b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
@@ -147,6 +147,18 @@ public void testCreateProduceProcessorsTextAndWildcard() {
 MediaType.APPLICATION_JSON + "," + MediaType.APPLICATION_XML + "," 
+ MediaType.WILDCARD));
   }
 
+  @Test
+  public void testCreateProduceProcessorsWithSemicolon() {
+RestOperationMeta operationMeta = new RestOperationMeta();
+operationMeta.produces = Arrays
+.asList(MediaType.TEXT_PLAIN + ";charset=UTF-8", 
MediaType.APPLICATION_JSON + ";charset=UTF-8");
+operationMeta.createProduceProcessors();
+Assert.assertSame(ProduceProcessorManager.PLAIN_PROCESSOR,
+operationMeta.ensureFindProduceProcessor(MediaType.TEXT_PLAIN));
+Assert.assertSame(ProduceProcessorManager.JSON_PROCESSOR,
+operationMeta.ensureFindProduceProcessor(MediaType.APPLICATION_JSON));
+  }
+
   @Test
   public void testEnsureFindProduceProcessorRequest(@Mocked 
HttpServletRequestEx requestEx) {
 RestOperationMeta operationMeta = new RestOperationMeta();
diff --git 
a/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/ConsumerMain.java
 
b/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/ConsumerMain.java
index 0ef503900..53edb8b9b 100644
--- 
a/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/ConsumerMain.java
+++ 
b/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/ConsumerMain.java
@@ -22,6 +22,7 @@
 import org.apache.servicecomb.it.deploy.MicroserviceDeploy;
 import org.apache.servicecomb.it.junit.ITJUnitUtils;
 import org.apache.servicecomb.it.schema.TestApiOperation;
+import org.apache.servicecomb.it.testcase.TestAcceptType;
 import org.apache.servicecomb.it.testcase.TestAnnotatedAttribute;
 import org.apache.servicecomb.it.testcase.TestApiParam;
 import org.apache.servicecomb.it.testcase.TestChangeTransport;
@@ -102,6 +103,7 @@ private static void runShareTestCases() throws Throwable {
 
 // only rest support default value feature
 ITJUnitUtils.runWithRest(TestDefaultValue.class);
+ITJUnitUtils.runWithRest(TestAcceptType.class);
 
 ITJUnitUtils.runWithRest(TestDownload.class);
 
diff --git 
a/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/testcase/TestAcceptType.java
 
b/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/testcase/TestAcceptType.java
new file mode 100644
index 0..9b2f2b18e
--- /dev/null
+++ 
b/integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/testcase/TestAcceptType.java
@@ -0,0 +1,102 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may 

[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-12-04 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16709403#comment-16709403
 ] 

ASF GitHub Bot commented on SCB-1051:
-

coveralls edited a comment on issue #1015: [SCB-1051]when interface set 
produces=text/plain;charset=utf-8. and c…
URL: 
https://github.com/apache/servicecomb-java-chassis/pull/1015#issuecomment-442428828
 
 
   
   [![Coverage 
Status](https://coveralls.io/builds/20445930/badge)](https://coveralls.io/builds/20445930)
   
   Coverage decreased (-0.003%) to 86.715% when pulling 
**eeaef6f635b87c4ff17bd77b72a608f9997a3ed7 on heyile:producerCharset** into 
**a21611163909bd075f3f7c737474e44c95f4c062 on apache:master**.
   


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


> when interface set produces=text/plain;charset=utf-8. and consumers set 
> accept = text/plain,will cause error
> 
>
> Key: SCB-1051
> URL: https://issues.apache.org/jira/browse/SCB-1051
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Java-Chassis
>Reporter: 何一乐
>Assignee: 何一乐
>Priority: Major
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-12-04 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708816#comment-16708816
 ] 

ASF GitHub Bot commented on SCB-1051:
-

heyile commented on a change in pull request #1015: [SCB-1051]when interface 
set produces=text/plain;charset=utf-8. and c…
URL: 
https://github.com/apache/servicecomb-java-chassis/pull/1015#discussion_r238695923
 
 

 ##
 File path: 
integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/testcase/TestAcceptType.java
 ##
 @@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.servicecomb.it.testcase;
+
+import org.apache.servicecomb.it.Consumers;
+import org.apache.servicecomb.swagger.invocation.exception.InvocationException;
+import org.junit.Assert;
+import org.junit.Test;
+import org.springframework.http.HttpEntity;
+import org.springframework.http.HttpHeaders;
+import org.springframework.http.HttpMethod;
+import org.springframework.http.MediaType;
+import org.springframework.http.ResponseEntity;
+
+public class TestAcceptType {
+  interface AcceptTypeIntf {
+  }
+
+  private static Consumers consumersAcceptTypeSpringmvc = new 
Consumers<>("acceptTypeSpringmvcSchema",
+  AcceptTypeIntf.class);
+
+  private static Consumers consumersAcceptTypeJaxrs = new 
Consumers<>("acceptTypeJaxrsSchema",
+  AcceptTypeIntf.class);
+
+
+  @Test
+  public void testTextPlain_rt() {
+checkTextPlain(consumersAcceptTypeSpringmvc);
+checkTextPlain(consumersAcceptTypeJaxrs);
+  }
+
+  private void checkTextPlain(Consumers consumers) {
+String result = textHeader_rt(consumers, MediaType.TEXT_PLAIN_VALUE);
+Assert.assertEquals("cse", result);
+
+try {
+  textHeader_rt(consumers, MediaType.APPLICATION_JSON_VALUE);
+  Assert.fail("should throw exception");
+} catch (InvocationException e) {
+  Assert.assertEquals(406, e.getStatusCode());
+  Assert.assertTrue(e.getMessage().contains("Accept application/json is 
not supported"));
+} catch (Exception e) {
 
 Review comment:
   ok


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


> when interface set produces=text/plain;charset=utf-8. and consumers set 
> accept = text/plain,will cause error
> 
>
> Key: SCB-1051
> URL: https://issues.apache.org/jira/browse/SCB-1051
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Java-Chassis
>Reporter: 何一乐
>Assignee: 何一乐
>Priority: Major
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-12-02 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16706756#comment-16706756
 ] 

ASF GitHub Bot commented on SCB-1051:
-

wujimin commented on a change in pull request #1015: [SCB-1051]when interface 
set produces=text/plain;charset=utf-8. and c…
URL: 
https://github.com/apache/servicecomb-java-chassis/pull/1015#discussion_r238158735
 
 

 ##
 File path: 
integration-tests/it-consumer/src/main/java/org/apache/servicecomb/it/testcase/TestAcceptType.java
 ##
 @@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.servicecomb.it.testcase;
+
+import org.apache.servicecomb.it.Consumers;
+import org.apache.servicecomb.swagger.invocation.exception.InvocationException;
+import org.junit.Assert;
+import org.junit.Test;
+import org.springframework.http.HttpEntity;
+import org.springframework.http.HttpHeaders;
+import org.springframework.http.HttpMethod;
+import org.springframework.http.MediaType;
+import org.springframework.http.ResponseEntity;
+
+public class TestAcceptType {
+  interface AcceptTypeIntf {
+  }
+
+  private static Consumers consumersAcceptTypeSpringmvc = new 
Consumers<>("acceptTypeSpringmvcSchema",
+  AcceptTypeIntf.class);
+
+  private static Consumers consumersAcceptTypeJaxrs = new 
Consumers<>("acceptTypeJaxrsSchema",
+  AcceptTypeIntf.class);
+
+
+  @Test
+  public void testTextPlain_rt() {
+checkTextPlain(consumersAcceptTypeSpringmvc);
+checkTextPlain(consumersAcceptTypeJaxrs);
+  }
+
+  private void checkTextPlain(Consumers consumers) {
+String result = textHeader_rt(consumers, MediaType.TEXT_PLAIN_VALUE);
+Assert.assertEquals("cse", result);
+
+try {
+  textHeader_rt(consumers, MediaType.APPLICATION_JSON_VALUE);
+  Assert.fail("should throw exception");
+} catch (InvocationException e) {
+  Assert.assertEquals(406, e.getStatusCode());
+  Assert.assertTrue(e.getMessage().contains("Accept application/json is 
not supported"));
+} catch (Exception e) {
 
 Review comment:
   change Exception to Throwable 
   and delete below useless empty lines.


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


> when interface set produces=text/plain;charset=utf-8. and consumers set 
> accept = text/plain,will cause error
> 
>
> Key: SCB-1051
> URL: https://issues.apache.org/jira/browse/SCB-1051
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Java-Chassis
>Reporter: 何一乐
>Assignee: 何一乐
>Priority: Major
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-11-28 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16701809#comment-16701809
 ] 

ASF GitHub Bot commented on SCB-1051:
-

coveralls commented on issue #1015: [SCB-1051]when interface set 
produces=text/plain;charset=utf-8. and c…
URL: 
https://github.com/apache/servicecomb-java-chassis/pull/1015#issuecomment-442428828
 
 
   
   [![Coverage 
Status](https://coveralls.io/builds/20334594/badge)](https://coveralls.io/builds/20334594)
   
   Coverage increased (+0.008%) to 86.782% when pulling 
**8740e6be79a1858b64f21d815746bdca73ece34b on heyile:producerCharset** into 
**47d4773f3db7444e07fa2bb5bf87ac0a0387855f on apache:master**.
   


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


> when interface set produces=text/plain;charset=utf-8. and consumers set 
> accept = text/plain,will cause error
> 
>
> Key: SCB-1051
> URL: https://issues.apache.org/jira/browse/SCB-1051
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Java-Chassis
>Reporter: 何一乐
>Assignee: 何一乐
>Priority: Major
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-11-28 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16701525#comment-16701525
 ] 

ASF GitHub Bot commented on SCB-1051:
-

heyile commented on a change in pull request #1015: [SCB-1051]when interface 
set produces=text/plain;charset=utf-8. and c…
URL: 
https://github.com/apache/servicecomb-java-chassis/pull/1015#discussion_r236980511
 
 

 ##
 File path: 
common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
 ##
 @@ -147,6 +147,18 @@ public void testCreateProduceProcessorsTextAndWildcard() {
 MediaType.APPLICATION_JSON + "," + MediaType.APPLICATION_XML + "," 
+ MediaType.WILDCARD));
   }
 
+  @Test
+  public void testCreateProduceProcessorsWithSemicolon() {
 
 Review comment:
   ok. l see


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


> when interface set produces=text/plain;charset=utf-8. and consumers set 
> accept = text/plain,will cause error
> 
>
> Key: SCB-1051
> URL: https://issues.apache.org/jira/browse/SCB-1051
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Java-Chassis
>Reporter: 何一乐
>Assignee: 何一乐
>Priority: Major
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-11-28 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16701521#comment-16701521
 ] 

ASF GitHub Bot commented on SCB-1051:
-

heyile commented on a change in pull request #1015: [SCB-1051]when interface 
set produces=text/plain;charset=utf-8. and c…
URL: 
https://github.com/apache/servicecomb-java-chassis/pull/1015#discussion_r236980603
 
 

 ##
 File path: 
common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
 ##
 @@ -147,6 +147,18 @@ public void testCreateProduceProcessorsTextAndWildcard() {
 MediaType.APPLICATION_JSON + "," + MediaType.APPLICATION_XML + "," 
+ MediaType.WILDCARD));
   }
 
+  @Test
+  public void testCreateProduceProcessorsWithSemicolon() {
 
 Review comment:
   ok, I see


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


> when interface set produces=text/plain;charset=utf-8. and consumers set 
> accept = text/plain,will cause error
> 
>
> Key: SCB-1051
> URL: https://issues.apache.org/jira/browse/SCB-1051
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Java-Chassis
>Reporter: 何一乐
>Assignee: 何一乐
>Priority: Major
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-11-28 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16701520#comment-16701520
 ] 

ASF GitHub Bot commented on SCB-1051:
-

heyile commented on a change in pull request #1015: [SCB-1051]when interface 
set produces=text/plain;charset=utf-8. and c…
URL: 
https://github.com/apache/servicecomb-java-chassis/pull/1015#discussion_r236980511
 
 

 ##
 File path: 
common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
 ##
 @@ -147,6 +147,18 @@ public void testCreateProduceProcessorsTextAndWildcard() {
 MediaType.APPLICATION_JSON + "," + MediaType.APPLICATION_XML + "," 
+ MediaType.WILDCARD));
   }
 
+  @Test
+  public void testCreateProduceProcessorsWithSemicolon() {
 
 Review comment:
   ok. let me see


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


> when interface set produces=text/plain;charset=utf-8. and consumers set 
> accept = text/plain,will cause error
> 
>
> Key: SCB-1051
> URL: https://issues.apache.org/jira/browse/SCB-1051
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Java-Chassis
>Reporter: 何一乐
>Assignee: 何一乐
>Priority: Major
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-11-28 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16701524#comment-16701524
 ] 

ASF GitHub Bot commented on SCB-1051:
-

heyile commented on a change in pull request #1015: [SCB-1051]when interface 
set produces=text/plain;charset=utf-8. and c…
URL: 
https://github.com/apache/servicecomb-java-chassis/pull/1015#discussion_r236980603
 
 

 ##
 File path: 
common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
 ##
 @@ -147,6 +147,18 @@ public void testCreateProduceProcessorsTextAndWildcard() {
 MediaType.APPLICATION_JSON + "," + MediaType.APPLICATION_XML + "," 
+ MediaType.WILDCARD));
   }
 
+  @Test
+  public void testCreateProduceProcessorsWithSemicolon() {
 
 Review comment:
   ok, I see


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


> when interface set produces=text/plain;charset=utf-8. and consumers set 
> accept = text/plain,will cause error
> 
>
> Key: SCB-1051
> URL: https://issues.apache.org/jira/browse/SCB-1051
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Java-Chassis
>Reporter: 何一乐
>Assignee: 何一乐
>Priority: Major
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-11-27 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16701368#comment-16701368
 ] 

ASF GitHub Bot commented on SCB-1051:
-

liubao68 commented on a change in pull request #1015: [SCB-1051]when interface 
set produces=text/plain;charset=utf-8. and c…
URL: 
https://github.com/apache/servicecomb-java-chassis/pull/1015#discussion_r236931110
 
 

 ##
 File path: 
common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
 ##
 @@ -147,6 +147,18 @@ public void testCreateProduceProcessorsTextAndWildcard() {
 MediaType.APPLICATION_JSON + "," + MediaType.APPLICATION_XML + "," 
+ MediaType.WILDCARD));
   }
 
+  @Test
+  public void testCreateProduceProcessorsWithSemicolon() {
 
 Review comment:
   Can you add an integration tests for this? This is an user case, add an 
integration tests is better. 


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


> when interface set produces=text/plain;charset=utf-8. and consumers set 
> accept = text/plain,will cause error
> 
>
> Key: SCB-1051
> URL: https://issues.apache.org/jira/browse/SCB-1051
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Java-Chassis
>Reporter: 何一乐
>Assignee: 何一乐
>Priority: Major
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SCB-1051) when interface set produces=text/plain;charset=utf-8. and consumers set accept = text/plain,will cause error

2018-11-27 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/SCB-1051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16700516#comment-16700516
 ] 

ASF GitHub Bot commented on SCB-1051:
-

heyile opened a new pull request #1015: [SCB-1051]when interface set 
produces=text/plain;charset=utf-8. and c…
URL: https://github.com/apache/servicecomb-java-chassis/pull/1015
 
 
   …onsumers set accept = text/plain,will cause error
   
   Follow this checklist to help us incorporate your contribution quickly and 
easily:
   
- [ ] Make sure there is a [JIRA 
issue](https://issues.apache.org/jira/browse/SCB) filed for the change (usually 
before you start working on it).  Trivial changes like typos do not require a 
JIRA issue.  Your pull request should address just this issue, without pulling 
in other changes.
- [ ] Each commit in the pull request should have a meaningful subject line 
and body.
- [ ] Format the pull request title like `[SCB-XXX] Fixes bug in 
ApproximateQuantiles`, where you replace `SCB-XXX` with the appropriate JIRA 
issue.
- [ ] Write a pull request description that is detailed enough to 
understand what the pull request does, how, and why.
- [ ] Run `mvn clean install` to make sure basic checks pass. A more 
thorough check will be performed on your pull request automatically.
- [ ] If this contribution is large, please file an Apache [Individual 
Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).
   
   ---
   


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


> when interface set produces=text/plain;charset=utf-8. and consumers set 
> accept = text/plain,will cause error
> 
>
> Key: SCB-1051
> URL: https://issues.apache.org/jira/browse/SCB-1051
> Project: Apache ServiceComb
>  Issue Type: Improvement
>  Components: Java-Chassis
>Reporter: 何一乐
>Assignee: 何一乐
>Priority: Major
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)