canonicalization bug - do not add '/' after '?', add it right after hostname if needed.

1 files changed, 9 insertions, 2 deletions

diff --git a/lib/consumer.cc b/lib/consumer.cc
index cbe0769..bb6358c 100644
--- a/lib/consumer.cc
+++ b/lib/consumer.cc
@@ -330,12 +330,19 @@ namespace opkele {
         if(i==string::npos) { // primitive. but do we need more?
             rv.insert(0,"http://");
             i = sizeof("http://")-1;
         }else{
             i += sizeof("://")-1;
         }
-        if(rv.find('/',i)==string::npos)
-            rv += '/';
+        string::size_type qm = rv.find('?',i);
+        string::size_type sl = rv.find('/',i);
+        if(qm!=string::npos) {
+            if(sl==string::npos || sl>qm)
+                rv.insert(qm,1,'/');
+        }else{
+            if(sl==string::npos)
+                rv += '/';
+        }
         return rv;
     }
 
 }