]> git.mxchange.org Git - quix0rs-gnu-social.git/commit
Avoid spewing giant debug backtrace into exception in certain OStatus subscription...
authorBrion Vibber <brion@pobox.com>
Wed, 5 May 2010 00:11:43 +0000 (17:11 -0700)
committerBrion Vibber <brion@pobox.com>
Wed, 5 May 2010 00:11:43 +0000 (17:11 -0700)
commit5f5d9518bdc5daec5a9250a852777d3015856c78
tree6e681a2c3cf535fa4f46fd3d5ad98bd89babc3f3
parent9d2c3d45bc32f54d325072d67d8426edd0e112f0
Avoid spewing giant debug backtrace into exception in certain OStatus subscription failure cases.

The code pattern 'new XXXException($e)' to chain exceptions doesn't actually work as intended, as exceptions are actually expecting a string message here.
This caused an implicit string conversion from HTTP_Request2_Exception, which is a PEAR_Exception, which defines an absurdly detailed __toString() method including a giant HTML table with a backtrace if you happen to be on a web request.
Simply passing $e->getMessage() instead clears this up, as we'll get the nice short message like 'Couldn't connect to tcp://blahblah:80'
plugins/OStatus/lib/feeddiscovery.php