When tracking request is > 2000 char, prefer POST (#11201)

* fixes #2321 when tracking request is > 2000 char, prefer POST * Fix jslint
matomo-org · Jan 17, 2017 · 6c0e31b · 6c0e31b
1 parent d807689
commit 6c0e31b
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/core/Tracker.php b/core/Tracker.php
@@ -76,7 +76,7 @@ private function init()
             ExceptionHandler::setUp();
 
             Common::printDebug("Debug enabled - Input parameters: ");
-            Common::printDebug(var_export($_GET, true));
+            Common::printDebug(var_export($_GET + $_POST, true));
         }
     }
 

diff --git a/js/piwik.js b/js/piwik.js
@@ -3528,7 +3528,7 @@ if (typeof window.Piwik !== 'object') {
             function sendRequest(request, delay, callback) {
                 if (!configDoNotTrack && request) {
                     makeSureThereIsAGapAfterFirstTrackingRequestToPreventMultipleVisitorCreation(function () {
-                        if (configRequestMethod === 'POST') {
+                        if (configRequestMethod === 'POST' || String(request).length > 2000) {
                             sendXmlHttpRequest(request, callback);
                         } else {
                             getImage(request, callback);

diff --git a/js/piwik.min.js b/js/piwik.min.js
diff --git a/piwik.js b/piwik.js