ignore hash fragment when parsing query string response URLs

brockallen · brockallen · commit 626a0d75cc3a · 2018-12-12T13:19:11.000-05:00
diff --git a/src/UrlUtility.js b/src/UrlUtility.js
@@ -31,6 +31,14 @@ export class UrlUtility {
             value = value.substr(idx + 1);
         }
 
+        if (delimiter === "?") {
+            // if we're doing query, then strip off hash fragment before we parse
+            idx = value.indexOf('#');
+            if (idx >= 0) {
+                value = value.substr(0, idx);
+            }
+        }
+
         var params = {},
             regex = /([^&=]+)=([^&]*)/g,
             m;
diff --git a/test/unit/UrlUtility.spec.js b/test/unit/UrlUtility.spec.js
@@ -45,6 +45,16 @@ describe("UrlUtility", function() {
             result.should.deep.equal({ a: "apple", b: "banana", c: "carrot" });
         });
 
+        it("should parse query string", function() {
+            let result = UrlUtility.parseUrlFragment("http://server?test1=xoxo&test2=yoyo", "?");
+            result.should.deep.equal({ test1: "xoxo", test2: "yoyo" });
+        });
+        
+        it("should parse query string up to hash", function() {
+            let result = UrlUtility.parseUrlFragment("http://server?test1=xoxo&test2=yoyo#a=apple&b=banana&c=carrot", "?");
+            result.should.deep.equal({ test1: "xoxo", test2: "yoyo" });
+        });
+
         it("should return error for long values", function() {
             let result = UrlUtility.parseUrlFragment("a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple&a=apple");
             result.should.have.property('error');
