Symfony2: Problems using logger in a service (passed as as parameter)

Question

I'm developing a project using Symfony 2.1.

I have created a service that is being called from a controller, and it's working ok. Now I need that the service generates log, and I'm trying to pass logger this way:

soap.client:
    class: MyFirm\MyAppBundle\Service\SOAPClient
        arguments: 
            logger: "@logger"

My service is defined this way:

namespace MyFirm\MyAppBundle\Service;

use \SoapClient as SoapClient;

use Monolog\Logger;

class SOAPClient
{
    private $logger;

    function __construct (Logger $logger)
    {
        $this->logger = $logger;
    }

    function sendMessage ($message, $wsdl_url) 
    {
        $webServResult = "ERROR";

        try{
            $client = new SoapClient($wsdl_url, array("trace"=>true,
                    "exceptions"=>true));        
            $webServResult=$client->sendMessage($data);
        }
        catch(\Exception $ex){
            $webServResult="ERROR";
            $message=$ex->getMessage();
            $log_text = print_r($ex, true)."\n".
                    $client->__getLastRequest()."\n".
                    $client->__getLastResponse();
            $this->logger->err("ERROR: ".$log_text);            
        }               

        return $webServResult;
    }


}

However, when I use the logger (if the wsdl doesn't exist, for example), the application hangs.

Am I doing anything wrong? Thanks a lot.

It at least helped me to use logger from the service. – BartBiczBoży Apr 16 '16 at 03:48 — BartBiczBoży, Apr 16 '16 at 03:48

score 0 · Answer 1 · answered Jan 17 '13 at 17:44

0

This is not a problzm from logger but from SoapClient that doesn't throw any Exception in case of unreachable WSDL... It waits until a Fatal error is thrown...

You have to check if WSDl exists before calling SoapClient ;)

answered Jan 17 '13 at 17:44

AlterPHP

12,667
5
49
54

SoapClient does generate an Exception if the WSDL is not found: **SOAP-ERROR: Parsing WSDL: Couldn't load from 'http://localhost/unknown.wsdl' : failed to load external entity "http://localhost/unknown.wsdl"** – DaveFX Jan 17 '13 at 17:53
OK, I had a far souvenir that it was not handled as a classic exception. – AlterPHP Jan 17 '13 at 18:04

score 0 · Accepted Answer · answered Jan 17 '13 at 17:52

Sorry, the problem wasn't at logger, but in the lines:

  $log_text = print_r($ex, true)."\n".
                $client->__getLastRequest()."\n".
                $client->__getLastResponse();

If I remove these lines and log only the Exception message all goes right. SoapClient does generate an Exception if the WSDL is not found:

SOAP-ERROR: Parsing WSDL: Couldn't load from 'http://localhost/unknown.wsdl' : 
             failed to load external entity "http://localhost/unknown.wsdl"

Solved and fixed. Sorry for the spam.

Symfony2: Problems using logger in a service (passed as as parameter)

2 Answers2