[
https://issues.apache.org/jira/browse/SCB-886?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16605199#comment-16605199
]
ASF GitHub Bot commented on SCB-886:
------------------------------------
coveralls edited a comment 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/18853006)
Coverage increased (+0.01%) to 86.063% when pulling
**c2d1d0455920ac97809ef600a56c7c2db76daeaf on
yhs0092:fix_url_encode_decode_problem** into
**36f897fd9facaead112f57e360993b0878e5d013 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)
