Bug #75215 [Nab]: set_exception_handler callable not called on exception

From: 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

« previous php.bugs (#211195) next »