[
https://issues.apache.org/jira/browse/SCB-886?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16602380#comment-16602380
]
ASF GitHub Bot commented on SCB-886:
------------------------------------
coveralls commented on issue #888: [SCB-886] Fix path param encode decode
problem
URL:
https://github.com/apache/incubator-servicecomb-java-chassis/pull/888#issuecomment-418169558
[](https://coveralls.io/builds/18806850)
Coverage increased (+0.02%) to 86.196% when pulling
**280cac2dd497858dbe5020edbacb5407155d9550 on
yhs0092:fix_url_encode_decode_problem** into
**be33086dbde190ceec425aeea3ae5662fbf6b633 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:
[email protected]
> Path param is not encoded and decoded correctly
> -----------------------------------------------
>
> Key: SCB-886
> URL: https://issues.apache.org/jira/browse/SCB-886
> Project: Apache ServiceComb
> Issue Type: Bug
> Components: Java-Chassis
> Reporter: YaoHaishi
> Assignee: YaoHaishi
> Priority: Major
>
> We find some problems:
> # for provider side, "%20" and "+" are both decoded to space char " ". But
> for path param, only "%20" should be decoded to space, and "+" should not be
> converted. See discussion in [QueryStringDecoder does not decode correctly
> path part with '+' (plus) sign in
> it|https://github.com/netty/netty/issues/6745]
> # for consumer side, currently path param is not encoded.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
