Merge pull request #22 from segmentio/fix/page-defaults-search

update page defaults search method

Author: William Johnson

lib/pageDefaults.js

@@ -20,7 +20,7 @@ function pageDefaults() {
   return {
     path: canonicalPath(),
     referrer: document.referrer,
-    search: location.search,
+    search: searchPath(),
     title: document.title,
     url: canonicalUrl(location.search)
   };
@@ -55,6 +55,21 @@ function canonicalUrl(search) {
   return i === -1 ? url : url.slice(0, i);
 }
 
+/**
+ * Return the search path for the page
+ * This is preferable to window.location.search because SPAs
+ * can have a # in the url which will include the query param in
+ * the hash
+ *
+ * @return {string}
+ */
+
+function searchPath() {
+  var url = window.location.href;
+  var u = url.indexOf('?');
+  return u === -1 ? '' : url.slice(u, -1);
+}
+
 /*
  * Exports.
  */