Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.I got annoyed looking at referrer reports that contained alternate domains (ie. for the site jeancrowder.ca, it wasn't identifying jeancrowder.org, jeancrowder.com or jeancrowder.net, let alone crowder.openconcept.ca as being internal references).
I wrote up a small patch for 4.6 & 4.7 that addresses this. Essentially there just needs to be some means to identify what domains should be considered internal and therefore removed from the query of external references.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | statistics.module.46.patch | 2.41 KB | mgifford |
| statistics.module.47.patch | 2.53 KB | mgifford | |
Comments
Comment #1
mgiffordJust adding the 4.6 patch.
Comment #2
StevenPatzFeatures go in cvs
Comment #3
bdragon CreditAttribution: bdragon commentedAutopatch Results:
patching file statistics.module
Hunk #1 succeeded at 354 (offset 43 lines).
Hunk #2 succeeded at 337 with fuzz 2.
Hunk #3 FAILED at 363.
1 out of 3 hunks FAILED -- saving rejects to file statistics.module.rej
Comment #4
PanchoGood point, we need this, but unfortunately not before D7.
Comment #5
PanchoComment #6
mgiffordWhy not? Or rather we should deal with this now to get it into D7?
Comment #7
mgiffordThis might just need to be closed. I haven't evaluated if it is still an issue or not.
Comment #13
Wim LeersThe Statistics module no longer looks at the
Refererheader, so this can't be a problem anymore.(Interestingly, #1694922: Send more data to the statistics backend is looking to restore that. IDK why it was removed.)