Bug #75215 [Nab]: set_exception_handler callable not called on exception
| From: | bkfake-php at yahoo dot com | Date: | Sun, 17 Sep 2017 15:05:52 +0000 |
| Subject: | Bug #75215 [Nab]: set_exception_handler callable not called on exception | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-211195@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=75215&edit=1
ID: 75215
User updated by: bkfake-php at yahoo dot com
Reported by: bkfake-php at yahoo dot com
Summary: set_exception_handler callable not called on
exception
Status: Not a bug
Type: Bug
Package: *General Issues
Operating System: OSX
PHP Version: 7.0.23
Block user comment: N
Private report: N
New Comment:
> When the exception is thrown the call stack begins unraveling: first myMethod, then doIt. But
> when doIt exits, there are no more references to $test so its destructor is called. That destructor
> exit()s, so normal execution flow ends and shutdown begins. The exception didn't make it to the
> top of the stack for it to be uncaught.
I would think that when the exception is throws, it should first go up the call stack trying to
catch the exception.
Only after the exception is caught (or missed) should it "deravel" the stack and begin
unreferencing the objects..
If that simply isn't possible, it seems that there should be a fatal error: "uncaught
exception"
Note: I'm not the bonehead that put an exit in a destructor.. I'm the monkey that tried
to find why the page was exiting without throwing any error or exception (as it appears).
Previous Comments:
------------------------------------------------------------------------
[2017-09-17 07:13:13] requinix@php.net
This behavior makes sense to me. Quoting the docs,
"The destructor method will be called as soon as there are no other references to a particular
object,"
When the exception is thrown the call stack begins unraveling: first myMethod, then doIt. But when
doIt exits, there are no more references to $test so its destructor is called. That destructor
exit()s, so normal execution flow ends and shutdown begins. The exception didn't make it to the
top of the stack for it to be uncaught.
"Calling exit() in a destructor will prevent the remaining shutdown routines from
executing."
As the sentence immediately before that says, this applies to when a destructor is invoked during
shutdown. That is not the case here - it's the opposite, in fact.
------------------------------------------------------------------------
[2017-09-17 03:45:14] bkfake-php at yahoo dot com
Description:
------------
exception handler is never called!
it appears to me that the exception itself causes the destruction of the object.. which calls exit.
this somehow bypasses the exception handler.
yet, shutdown function is called!
also note: error_get_last() inside the shutdown function returns null.. this is usually where you
can catch fatal error (including the fatal error: "uncaught exception").
Here the exception was completely swallowed and never reported
PHP's OOP destructor documentation includes this brief
"Calling exit() in a destructor will prevent the remaining shutdown routines from
executing."
a) shutdown function was called, which seems to contradict documentation
b) no mention of of exception handler... which seems like it should have been called before the
destruction of the objects!
same behavior in 5.6 and 7.0
Test script:
---------------
<?php
class Test
{
public function myMethod()
{
echo __METHOD__.'<br />';
echo 'throwing exception<br />';
throw new Exception('hello world?');
}
public function __destruct()
{
echo __METHOD__.'<br />';
exit;
}
}
class Wrapper
{
public function doIt()
{
$test = new Test();
$test->myMethod();
}
public function __destruct()
{
echo __METHOD__.'<br />';
}
}
register_shutdown_function(function () {
echo 'shutdown function<br />';
echo 'error_get_last: '.json_encode(error_get_last()).'<br />';
});
set_exception_handler(function ($e) {
echo 'exception handler: '.$e->getMessage().'<br />';
});
echo 'Main<br />';
$wrapper = new Wrapper();
$wrapper->doIt();
Expected result:
----------------
exception handler should be called?!
output should include "exception handler: hello world?"
Actual result:
--------------
exception not caught. exception "swallowed" and script exits sans error with http 200
Test script outputs
Main
Test::myMethod
throwing exception
Test::__destruct
shutdown function
error_get_last: null
Wrapper::__destruct
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=75215&edit=1