Project

General

Profile

Actions

Task #39347

closed

Remove page hit statistics

Added by Mario Rimann over 12 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
-
Start date:
2012-07-28
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

For TYPO3 6.0, the included logging of each single page request should be removed in favor of using the well known tools like Google Analytics or Piwik.


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #52786: Remove TYPO3_CONF_VARS['FE']['logfile_dir']Closed2013-10-14

Actions
Related to TYPO3 Core - Task #56849: Remove deprecated functions from TypoScriptFrontendControllerClosed2014-03-12

Actions
Related to TYPO3 Core - Bug #22529: stat_apache always logs HTTP status 200Rejected2010-04-27

Actions
Actions #1

Updated by Gerrit Code Review over 12 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229

Actions #2

Updated by Gerrit Code Review over 12 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229

Actions #3

Updated by Gerrit Code Review over 12 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229

Actions #4

Updated by Gerrit Code Review over 12 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229

Actions #5

Updated by Gerrit Code Review over 12 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229

Actions #6

Updated by Christian Kuhn over 12 years ago

created documentation issue #39468

Actions #7

Updated by Gerrit Code Review over 12 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13229

Actions #8

Updated by Anonymous over 12 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #9

Updated by Bernhard Eckl over 11 years ago

So it is not possible to use statistic extension like ics_awstats anymore :-(
I think many people use this!

Actions #10

Updated by Bernhard Eckl over 11 years ago

In 4.7 config.stat_IP_anonymize and so on was added an in 6.0 everything removed?

Actions #11

Updated by Chris topher over 11 years ago

Bernhard Eckl wrote:

I think many people use this!

used this. ;-)

Actions #12

Updated by Clemens Riccabona about 11 years ago

Painful. Just painful.
I wonder sometimes, who makes decisions of such momentousness.
Ok, i read the core mailinglist, but not extra-carefully. Maybe I should change this behaviour. I did not recognize that there was made such a decision.

This now just means: now I have to configure statfile-writing via the webservers (apache in my case) vhost-files for each domain, configured in TYPO3, just to get awstats (e.g. ics_awstats) running.
before i could do it within TYPO3, where the statistics is presented to the author anyway.
Painful, really painful.

Actions #13

Updated by Christian Kuhn about 11 years ago

Extension like awstats could adapt and use some hook in the frontend to implement the logging on their own.

Actions #14

Updated by Clemens Riccabona about 11 years ago

Christian Kuhn wrote:

Extension like awstats could adapt and use some hook in the frontend to implement the logging on their own.

does it make sense, that functionality, which worked (at least for me) now about 10 (TEN) years, to remove from core and say: oh, if you want, you may hook.
nice to know that there is a hooking possibility for writing logfiles anyway.

but to me it does not make any sense.
was the code for writing logfiles in apache logformat so painful for the core developers, that it made sense for them to remove?
as i already stated, i did not recognize any discussion about that issue, i just stumbled upon luckily, today.
Therefore i am really interested in knowing why it is how it is now.

Actions #15

Updated by Christian Kuhn about 11 years ago

Yes, the code was so ugly, spaghetti, old, unmaintained, evil and cluttered that it actively denied refactoring of more important stuff.
Sometimes it is better to hand over some stuff to a dedicated extension that can handle this better and without a dependency to core code. We decided to do so in this very special case.

Actions #16

Updated by Clemens Riccabona about 11 years ago

Thanks Christian, for clarification of why this decision was made.
I am also interested in how I could prevent missing such important, feature-breaking decisions for the future, as I already told you, I stumbled upon this issue more or less by accident. (and i had great luck sticking strictly to LTS versions in the production. I don't want to think about what would had happened if I updated all my TYPO3 installations without knowing this fact!)

So my next feature-wish (as a paying assoc. member) is: please make a system extension which re-enables writing log-files in well documented and wide spread apache-log-format. ;) ;)
Maybe we could find some sponsors?

(yeah, i know, the official name of the log format is "NCSA Common log format", but most people I think know it by its "nickname").

Actions #17

Updated by Mario Rimann about 11 years ago

Hi Clemens

I agree that the removal of this old feature could be troublesome for a small percentage of users of TYPO3. When we discussed this issue, many votes were towards removing it, as most of the participants of that discussion mentioned that they anyway use a specialized tool like e.g. Piwik or Google Analytics (which provides way more information for the marketing team rather than pure hit counting that TYPO3 could log). So it seemed like the better way to remove it entirely - instead of carrying along old and hard-to-maintain code any further. And the 6.0 Release seemed to be the perfect moment for it - being a version that was clearly mentioned to contain breaking changes.

Regarding how to keep up to date, I agree that's not that easy - because there are just many changes between two versions - which is basically a good sign that there's something evolving.

If you look at the commit message of this change, you can clearly see it's marked with a bunch of exclamation marks and the "tag" "-FEATURE" - which should read: "Attention, breaking change of type feature removal". (https://review.typo3.org/#/c/13229/6//COMMIT_MSG)

I suggest you have a look at the change log which is part of each version's release notes. You don't need to read through all lines, but rather scan for lines with a "[!!!]" prefix - usually those are the things which will "break" something - or modify stuff in a way that needs you to adapt your previous work. Let's have a look at the Release Notes of 6.0.0 at http://wiki.typo3.org/TYPO3_6.0

This issue here was about removing the feature. If you'd like to get the feature back, please create a new issue and mention this one - and start a discussion on the dev- or core-mailinglist. This issue here is IMHO the wrong place for this discussion.

Best wishes,
Mario

Actions #18

Updated by Dan Kleine (Untenzu) almost 11 years ago

Clemens Riccabona wrote:

please make a system extension which re-enables writing log-files in well documented and wide spread apache-log-format

Hey Clemens, as Christian wrote, this should be part of a regular extension (and no system extension, since this is the same as having the code in the core again). Please create s Github Repository for such an extension and share it with others (maybe on a Forge Project?). Something like »awstats_logging« or »apache-style-logging«.

When this feature was removed I ported the removed logging code from the core into a hook of some AWStats extension, because I had to upgrade a project and support AWStats at this point of time. You can see the code here: https://bitbucket.org/pixelbrackets/typo3-xf_awstats/branch/writeLogfile
As proposed, this logging feature should be a single extension, without the AWStats analysis. I just dont want to maintain such an extension, thats why I did not put any effort in releasing it in a single extension.

As Mario states, further discussion should be made in a feature ticket.

Actions #19

Updated by Loek Hilgersom over 9 years ago

Just a note for anyone who still wants to use this feature, knowing that it comes with a performance penalty:

I created a little extension which copies the exact functionality for writing the Apache-style log files as used to be part of the core until v4.7.
The only issue to take care of is to add the [FE][logfile_dir] setting in AdditionalConfiguration.php instead of LocalConfiguration because the Install tool will remove it otherwise.

http://typo3.org/extensions/repository/view/retrostats

Now the feature is no longer lost, it has only become optional, all for the better :-)

Actions #20

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF