[GitHub] cloudstack-docs pull request: Add note in dev.rst about use of the...

2016-04-08 Thread phillipkent
Github user phillipkent commented on the pull request: https://github.com/apache/cloudstack-docs/pull/17#issuecomment-207598302 @bhaisaab Just to clarify, the problem isn't when * is between other characters, it is a * anywhere in the request URL which will lead to disagreement of the

[GitHub] cloudstack-docs pull request: Add note in dev.rst about use of the...

2016-04-08 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack-docs/pull/17#issuecomment-207517469 I've also find this to be an issue with Marvin, and the same fix works with Marvin as well. I'll be send that PR soon. --- If your project is set up for it, yo

[GitHub] cloudstack-docs pull request: Add note in dev.rst about use of the...

2016-04-08 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack-docs/pull/17#issuecomment-207517322 LGTM, I tested this against ACS 4.5 whenever an API arg (of type string) has an asterisk between text (something like arg="some*value"). When @phillipkent origi

[GitHub] cloudstack-docs pull request: Add note in dev.rst about use of the...

2016-04-08 Thread phillipkent
GitHub user phillipkent opened a pull request: https://github.com/apache/cloudstack-docs/pull/17 Add note in dev.rst about use of the safe option in urllib.quote_plus() There is a mismatch between the URL encoding behavior of Python and CloudStack's internal URL encoder (Java URLEnc