You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@servicecomb.apache.org by GitBox <gi...@apache.org> on 2021/12/13 03:59:51 UTC

[GitHub] [servicecomb-java-chassis] yhs0092 opened a new issue #2662: Java-Chassis 1.3 版本consumer端path参数编码存在问题, 不支持特殊字符`;.,`

yhs0092 opened a new issue #2662:
URL: https://github.com/apache/servicecomb-java-chassis/issues/2662


   复现用例见截图(直接在`org.apache.servicecomb.common.rest.definition.path.PathVarParamWriter`类里加main函数):
   ![image](https://user-images.githubusercontent.com/22049770/145749407-d4016002-a076-4b85-aa49-3b16b4f80aad.png)
   
   Java-Chassis 1.3 版本没有正确对 `;` 和 `.` 以及 `,` 三种字符进行转义处理


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@servicecomb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [servicecomb-java-chassis] liubao68 commented on issue #2662: Java-Chassis 1.3 版本consumer端path参数编码存在问题, 不支持特殊字符`;.,`

Posted by GitBox <gi...@apache.org>.

liubao68 commented on issue #2662:
URL: https://github.com/apache/servicecomb-java-chassis/issues/2662#issuecomment-1006361184


   这个问题看起来属于理解错误。 `;`, `.`, `,` 是合法的 [URL-path-segment string](https://url.spec.whatwg.org/#syntax-url-path-segment) ， 是不需要转义的。 
   
   理解错误的关键点在于  `;`, `.`, `,`  在 path-segment 中是有特殊含义的，  使用者不期望表现出这个特殊含义。 作为框架处理层， 是不知道使用者的意图， 是表现特殊含义还是不表现特殊含义？ 提这个问题显然是期望不表现特殊含义。 
   
   考虑到上述歧义， 框架处理层只能二选一。 当使用者不期望表现出这个特殊含义时，需要自己预先转义，而不是框架转义。 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@servicecomb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org