Bug #51507

Use logging framework in DatabaseConnection

Added by Sascha Egerer almost 6 years ago. Updated 10 months ago.

Status:
Accepted
Priority:
Should have
Assignee:
Category:
Database API (Doctrine DBAL)
Start date:
2013-08-29
Due date:
% Done:

0%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
medium
Is Regression:
No
Sprint Focus:

Description

Currently the logging in DatabaseConnection is not really clean and complete.

DatabaseConnection should use the logging framework to log executed Queries and SQL errors.

History

#1 Updated by Philipp Gampe almost 6 years ago

  • Status changed from New to Needs Feedback
  • Target version deleted (6.2.0)
  • Complexity set to medium

A good idea and a nice and isolated way to start the integration of the logging framework.

Can you create a first patch and upload it to gerrit? http://wiki.typo3.org/CWT

#2 Updated by Sascha Egerer almost 6 years ago

I'm working on it...

#3 Updated by Gerrit Code Review almost 6 years ago

  • Status changed from Needs Feedback to Under Review

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

#4 Updated by Gerrit Code Review almost 6 years ago

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

#5 Updated by Gerrit Code Review almost 6 years ago

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

#6 Updated by Gerrit Code Review almost 6 years ago

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

#7 Updated by Gerrit Code Review almost 6 years ago

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

#8 Updated by Gerrit Code Review almost 6 years ago

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

#9 Updated by Gerrit Code Review almost 6 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23462

#10 Updated by Gerrit Code Review almost 6 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23462

#11 Updated by Gerrit Code Review almost 6 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23462

#12 Updated by Gerrit Code Review over 5 years ago

Patch set 10 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23462

#13 Updated by Gerrit Code Review over 5 years ago

Patch set 11 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23462

#14 Updated by Gerrit Code Review over 5 years ago

Patch set 12 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23462

#15 Updated by Gerrit Code Review over 5 years ago

Patch set 13 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23462

#16 Updated by Gerrit Code Review over 5 years ago

Patch set 14 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23462

#17 Updated by Gerrit Code Review over 5 years ago

Patch set 15 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23462

#18 Updated by Gerrit Code Review almost 5 years ago

Patch set 16 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23462

#19 Updated by Gerrit Code Review almost 5 years ago

Patch set 17 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23462

#20 Updated by Sascha Egerer almost 5 years ago

  • Is Regression set to No

Example logging framework configuration.

Put this into you AdditionalConfiguration.php. But do this never ever on any production system!

$GLOBALS['TYPO3_CONF_VARS']['LOG']['TYPO3']['CMS']['Core']['Database'] = array(
    'writerConfiguration' => array(
        \TYPO3\CMS\Core\Log\LogLevel::DEBUG => array(
            'TYPO3\\CMS\\Core\\Log\\Writer\\FileWriter' => array(
                // configuration for the writer
                'logFile' => 'typo3temp/logs/databaseQueries.log'
            )
        ),
    )
);

#21 Updated by Gerrit Code Review almost 5 years ago

Patch set 18 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23462

#22 Updated by Gerrit Code Review almost 5 years ago

Patch set 19 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23462

#23 Updated by Gerrit Code Review almost 5 years ago

Patch set 20 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23462

#24 Updated by Gerrit Code Review almost 5 years ago

Patch set 21 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/23462

#25 Updated by Gerrit Code Review almost 5 years ago

Patch set 22 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/23462

#26 Updated by Gerrit Code Review almost 5 years ago

Patch set 23 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/23462

#27 Updated by Gerrit Code Review almost 5 years ago

Patch set 24 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/23462

#28 Updated by Gerrit Code Review almost 5 years ago

Patch set 25 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/23462

#29 Updated by Gerrit Code Review over 4 years ago

Patch set 26 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/23462

#30 Updated by Gerrit Code Review over 4 years ago

Patch set 27 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/23462

#31 Updated by Gerrit Code Review over 4 years ago

Patch set 28 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/23462

#32 Updated by Gerrit Code Review over 4 years ago

Patch set 29 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/23462

#33 Updated by Gerrit Code Review about 4 years ago

Patch set 30 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/23462

#34 Updated by Christian Kuhn almost 4 years ago

  • Status changed from Under Review to On Hold

patch abandoned, solution "on hold" until logging decisions have been taken.

#35 Updated by Benni Mack over 1 year ago

  • Status changed from On Hold to Accepted
  • Target version set to 9 LTS

Now that we streamlined logging in most places in the core, Logging could be added to Doctrine DBAL

#36 Updated by Susanne Moog 10 months ago

  • Target version changed from 9 LTS to Candidate for patchlevel

Also available in: Atom PDF