Feature #37360

Catch PDO exceptions to avoid outputting database information in exception backtrace

Added by Aske Ertmann about 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
Should have
Assignee:
Category:
Persistence
Start date:
2012-05-21
Due date:
% Done:

100%

Estimated time:
PHP Version:
5.3
Has patch:
Yes
Complexity:
easy

Description

If the PDO connection throws an exception, it's not catched by FLOW3, which leads to a debug backtrace showing the calls to the PDO calls revealing the database information. Even though this only happens in Production context, I still believe that it should not be shown there.

I've created a patch and provided before and after screenshots. I have also testet that normal sql exceptions are not catching by this, e.g. a missing table.


Files

before.jpg (115 KB) before.jpg Aske Ertmann, 2012-05-21 14:55
after.jpg (147 KB) after.jpg Aske Ertmann, 2012-05-21 14:55
#1

Updated by Gerrit Code Review about 9 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/11369

#2

Updated by Gerrit Code Review about 9 years ago

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

#3

Updated by Gerrit Code Review about 9 years ago

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

#4

Updated by Gerrit Code Review about 9 years ago

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

#5

Updated by Gerrit Code Review about 9 years ago

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

#6

Updated by Gerrit Code Review about 9 years ago

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

#7

Updated by Karsten Dambekalns about 9 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF