changed RequestValidator updatePort() to use raw Query, raw Path and …

…raw Fragment

src/main/java/com/twilio/security/RequestValidator.java

@@ -141,9 +141,9 @@ private String updatePort(URI url, int newPort) {
                 url.getUserInfo(),
                 url.getHost(), 
                 newPort,
-                url.getPath(),
-                url.getQuery(),
-                url.getFragment()).toString();
+                url.getRawPath(),
+                url.getRawQuery(),
+                url.getRawFragment()).toString();
         } catch (Exception e) {
             return url.toString();
         }

src/test/java/com/twilio/security/RequestValidatorTest.java

@@ -114,4 +114,19 @@ public void testValidateAddsPortHttp() {
         Assert.assertTrue("Validator did not add port 80 to http url", isValid);
     }
 
+    @Test
+    public void testValidateRemovesPortHttpsWithSpecialParams() {
+        String url = "https://mycompany.com/myapp.php";
+        String param = "?param1=client%3AAnonymous";
+        url = url + param;
+        boolean isValid = validator.validate(url, params, "iQIea/lNtwezVhrpQ/KiGTEmJDA=");
+
+        Assert.assertTrue("Validator did not strip port from url", isValid);
+
+        url = url.replace(".com", ".com:1234");
+        isValid = validator.validate(url, params, "g7dthJtIvMAeMJ8XhiywDKG63Gg=");
+
+        Assert.assertTrue("Validator did not strip port from url", isValid);
+    }
+
 }