[
https://issues.apache.org/jira/browse/SCB-886?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16602738#comment-16602738
]
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/18813799)
Coverage increased (+0.04%) to 86.213% when pulling
**70194b853c4c02a6222c1caf9775937d2688c230 on
yhs0092:fix_url_encode_decode_problem** into
**8be5c34b87716111a917908e532e1213e0b6ac3a 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)
