[qpid-proton] branch master updated: PROTON-2074: [C++] Improve list of url encoded characters - '%' characters in user/passwords would behave badly!

[as...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

astitcher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/qpid-proton.git


The following commit(s) were added to refs/heads/master by this push:
     new a7ced83  PROTON-2074: [C++] Improve list of url encoded characters - '%' characters in user/passwords would behave badly!
a7ced83 is described below

commit a7ced834d3040f0292a8a99408d5569b0d3f63d8
Author: Andrew Stitcher <as...@apache.org>
AuthorDate: Thu Jul 18 13:27:15 2019 -0400

    PROTON-2074: [C++] Improve list of url encoded characters
    - '%' characters in user/passwords would behave badly!
---
 cpp/src/url.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/url.cpp b/cpp/src/url.cpp
index 7609c30..0320388 100644
--- a/cpp/src/url.cpp
+++ b/cpp/src/url.cpp
@@ -35,7 +35,7 @@ namespace {
 
 /** URL-encode src and append to dst. */
 static std::string pni_urlencode(const std::string &src) {
-    static const char *bad = "@:/";
+    static const char *bad = "@:/%[]?#";
 
     std::ostringstream dst;
     dst << std::hex << std::uppercase << std::setfill('0');


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org