Edit report at http://bugs.php.net/bug.php?id=42214&edit=1

 ID:               42214
 Comment by:       webmaster at guestwho dot com
 Reported by:      stuart dot caie at gmail dot com
 Summary:          SoapServer sends clients internal PHP errors
 Status:           Closed
 Type:             Bug
 Package:          SOAP related
 Operating System: Ubuntu
 PHP Version:      5.2.4RC1
 Assigned To:      dmitry

 New Comment:

Is it at least possible to add filename and error number to error
messages ? I often run into "call to member function on non-object" and
I have a dozen possible classes that this could occur in. The soapfaul
generated by PHP doesn't contain filename nor line number, so I'm flying
blind (no way to debug php fatal errors that happen during a SOAP call).


Previous Comments:
------------------------------------------------------------------------
[2007-09-05 11:23:29] dmi...@php.net

Fixed in CVS HEAD and PHP_5_2.

Now you can disable to send errors back to client.



$server = new SoapServer(NULL, 

    array('uri' => 'http://localhost/server.php',

          'send_errors' => false));



------------------------------------------------------------------------
[2007-09-03 10:56:21] j...@php.net

Usually when PHP hits a fatal error (that means _FATAL_ :) it's in such
state that continuing script run would be causing e.g. crash or the
engine is about to blow up in some other way. Don't bother submitting
bug report about that, it will not change. (I think there's already one
about it too..)



Assigned back to Dmitry.

------------------------------------------------------------------------
[2007-08-31 15:11:38] stuart dot caie at gmail dot com

Yes, I would like the soap extension to let me hide fatal error
messages. Please add that option, either in code or in the php.ini





I still think PHP itself needs to be fixed so it can continue running
user code after a fatal error. However, that is a bigger change and less
likely to be done. I'll look to see if such a bug is already raised,
otherwise I'll raise that as another bug.



Thanks

Stuart

------------------------------------------------------------------------
[2007-08-31 12:44:10] dmi...@php.net

PHP is not able to execute user-code after a fatal error.

The only thing I can do with this - provide a special SoapServer option
to hide error messages. In this case it always will send
"[SOAP-ENV:Server] Internal error".



Do you like such solution?

------------------------------------------------------------------------
[2007-08-05 21:58:46] stuart dot caie at gmail dot com

Description:
------------
When presenting a SOAP API to the general public, I do not want the text
of PHP errors to be send down the wire as "SOAP-ENV;Server" faults. It's
just as embarrassing and as much of a security risk as having the
display_errors INI option turned on - it could reveal exploitable
private implementation details to hostile users.



I would like to catch all PHP errors, log them and instead send the user
a custom SOAP fault which gives them a unique error ID to report (which
matches with my log), but does not reveal the actual PHP error message.



However,



1. use_soap_error_handler() does nothing. Set it to true, it sends out
SOAP-ENV:Server faults with the PHP error message. Set it to false, it
still sends out SOAP-ENV:Server faults with the PHP error message.





2. User-defined error handlers can't catch E_ERROR, E_PARSE,
E_CORE_ERROR, E_CORE_WARNING, E_COMPILE_ERROR, E_COMPILE_WARNING, and
most of E_STRICT. Other scripting languages such as Perl (via
$SIG{__DIE__}) or Ruby (via begin/rescue) let user code catch fatal
errors, PHP comes up short.



I'd like you to allow PHP to catch fatal errors in the user defined
error handler. If you won't fix that, please add some kind of kludge to
SoapServer so that it doesn't reveal the text of PHP errors to clients.

Reproduce code:
---------------
<?php // server.php

class test {

    function test() { obvious_error(); } // will cause an error

}

function error_handler($level, $error, $file, $line, $context) {

    $ticket = date('YmdHis-') . $_SERVER['REMOTE_ADDR'];

    if ($fh = fopen('/tmp/soap_error_log', 'a')) { fwrite($fh,
"[$ticket] $level: $error at $file line $line\n"); fclose($fh); }

    if (isset($server)) $server->fault('error', "report \"$ticket\" to
support");

}

set_error_handler('error_handler');

use_soap_error_handler(false);

$server = new SoapServer(NULL, array('uri' =>
'http://localhost/server.php'));

$server->setClass('test');

$server->handle();

?>



<?php // client.php

$client = new SoapClient(NULL, array('uri' =>
'http://localhost/server.php', 'location' =>
'http://localhost/server.php'));

$client ->test();

?>





Expected result:
----------------
client.php: Uncaught SoapFault exception: [error] report "<unique id>"
to support



server.php: entry in /tmp/soap_error_log reading:

[<unique id>] 1: Call to undefined function obvious_error() in
server.php line 4



Actual result:
--------------
client.php: Uncaught SoapFault exception: [SOAP-ENV:Server] Call to
undefined function obvious_error()



server.php: no entry in /tmp/soap_error_log.




------------------------------------------------------------------------



-- 
Edit this bug report at http://bugs.php.net/bug.php?id=42214&edit=1

Reply via email to