diff options
author | Peter Zhang <peter@innocraft.com> | 2022-04-11 16:10:53 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-04-11 16:10:53 +0300 |
commit | 8a8b51243641b08064a1b57813da711e96e76298 (patch) | |
tree | 186ae164a68f2a5de7f000e0e4645ef064f5638a /core/Tracker/Request.php | |
parent | 793c7db0bec494d7165c60e04cbf4db5f701419b (diff) |
[Bug]fix prefilght cors OPTIONS request record in the action visits (#19030)
* extend request with options and method
when options header and method is options do not record in the database.
* update function
update function
* update tests
update tests
* update tests
adjust code only trigger on option request
* remove class variable
remove server
* Update Request.php
add check request method
* drop option request
drop prefight request
* update reset
update reset
* return 204 on prefight
return 204 on prefight
* Update Tracker.php
accept cors
* fix typo & add type hint
* Update core/Tracker/RequestSet.php
* apply PSR12 code formatting
* adds test
Co-authored-by: sgiehl <stefan@matomo.org>
Diffstat (limited to 'core/Tracker/Request.php')
-rw-r--r-- | core/Tracker/Request.php | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/core/Tracker/Request.php b/core/Tracker/Request.php index 58911a25a5..2b1496f451 100644 --- a/core/Tracker/Request.php +++ b/core/Tracker/Request.php @@ -44,6 +44,8 @@ class Request protected $tokenAuth; + + /** * Stores plugin specific tracking request metadata. RequestProcessors can store * whatever they want in this array, and other RequestProcessors can modify these @@ -72,6 +74,7 @@ class Request $this->timestamp = time(); $this->isEmptyRequest = empty($params); + // When the 'url' and referrer url parameter are not given, we might be in the 'Simple Image Tracker' mode. // The URL can default to the Referrer, which will be in this case // the URL of the page containing the Simple Image beacon @@ -921,4 +924,5 @@ class Request } return false; } + } |