Bug #33366 [Com]: __soapCall Produces Incorrect Request

From: Date: Wed, 09 Apr 2014 13:40:34 +0000
Subject: Bug #33366 [Com]: __soapCall Produces Incorrect Request
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-185156@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=33366&edit=1

 ID:                 33366
 Comment by:         wtf dot stomp at gmail dot com
 Reported by:        cmantunes at gmail dot com
 Summary:            __soapCall Produces Incorrect Request
 Status:             Not a bug
 Type:               Bug
 Package:            SOAP related
 Operating System:   Debian
 PHP Version:        5.1.0
 Assigned To:        dmitry
 Block user comment: N
 Private report:     N

 New Comment:

This bug described event there: http://www.php.net/manual/en/soapclient.soapcall.php#57032


Previous Comments:
------------------------------------------------------------------------
[2014-04-09 13:34:02] wtf dot stomp at gmail dot com

Fix it please!

------------------------------------------------------------------------
[2011-07-30 14:31:18] jonasraoni at gmail dot com

Here are the fixes I had to make in order to get it working for *my* needs. I believe it will help
the other users that had this problem (I found many cases on internet, but no fixes).

class FixedSOAPClient extends SoapClient{
	private $method;
	private $argumentCount;
	/*
	Loading the WSDL through PHP instead of letting the SoapClient do this job, avoids breaking Apache.
I noticed it breaks just here, while loading the wsdl through *HTTPS*.
	Note: I believe that the __doRequest method should also be called when loading the .wsdl
	*/
	public function __construct($url){
		$s = file_get_contents($url);
		file_put_contents($url = md5($url) . '.wsdl', $s);
		parent::__construct($url, array(
			'compression' => SOAP_COMPRESSION_ACCEPT | SOAP_COMPRESSION_GZIP
		));
	}
	public function __call($function, $arguments){
		$this->argumentCount = count($arguments);
		/*
		Adding a bogus parameter to the beginning, since the SoapClient is "eating" the first
argument.
		*/
		array_unshift($arguments, 0);
		return parent::__call($this->method = $function, $arguments);
	}
	public function __doRequest($request, $location, $action, $version, $oneWay = 0){
		$xml = new DOMDocument('1.0', 'utf-8');
		$xml->loadXML($request);
		$d = $xml->documentElement;
		/*
		Placing the "lost" arguments inside the function node, their right place.
		*/
		for($o = $d->getElementsByTagName($this->method)->item(0); $o->nextSibling;
$o->appendChild($o->nextSibling));

		$xml = $xml->saveXML();
		/*
		The operation expected parameters to be named as arg1, arg2, insted of what PHP built, which was
param1, param2...
		*/
		if($this->argumentCount)
			foreach(range($this->argumentCount, 0) as $i)
				$xml = str_replace('param' . ($i + 1), 'arg' . $i, $xml);
		/*
		Removing boundary from the XML result, this must be part of a standard as the calls works fine on
other tools, the SoapClient should be able to handle it.
		*/
		$s = preg_replace('/--.*?--$/', '',
preg_replace('/^(?:.|\n|\r)*?<soap:/', '<soap:', parent::__doRequest($xml,
$location, $action, $version, $oneWay)));
		return $s;
	}
}

------------------------------------------------------------------------
[2011-07-15 21:27:34] jonasraoni at gmail dot com

I have the same problem as "cmantunes" (I'm using an WSDL too), and many more at the
same request.

#1. First argument being ripped off from the XML when calling directly ($client->method(args))

#2. After adding a bogus first parameter, the request got the method name closed and, the param was
placed after it, just like the guy mentioned. And, also with the wrong name, the param should be
"arg0" and it was displayed as "param1".

#3. Even after fixing the request, the response is also misunderstood by the extension! I had to
make more workarounds to remove the trailing data (which seems to be part of a standard, the request
works fine when I tested with the soapUI client):

--uuid:6365a322-9529-4cf5-b3bb-4ed69bd78b38
Content-Type: application/xop+xml; charset=UTF-8; type="text/xml";
Content-Transfer-Encoding: binary
Content-ID: <root.message@cxf.apache.org>

<soap:Envelope xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/">
[...]
--uuid:6365a322-9529-4cf5-b3bb-4ed69bd78b38--


#4. When using Apache 2.2.X on Windows and requesting a WSDL through HTTPS "new
SoapClient" crashes Apache. It works fine just on IIS, I tested with several PHP versions and,
also the latest (currently 5.3.6).

I had tried the suggested code by Dmitry before and it didn't work out.

Unhappily I made workarounds at the __doRequest to solve my immediate problems.

------------------------------------------------------------------------
[2005-06-17 10:03:33] dmitry@php.net

Hi,

The ext/soap is match easy in usage then you expected.
You should use the following code:

$isc->getOperationCount(array(
    'startDate' => '2005-04-01',
    'endDate'   => '2005-04-01'
  ));

or 

$isc->__soapCall('getOperationCount', array(
    array('getOperationCount' => array(
      'startDate' => '2005-04-01',
      'endDate'   => '2005-04-01'
  )));

You don't need to use SoapParam if you use WSDL.

------------------------------------------------------------------------
[2005-06-16 20:33:24] cmantunes at gmail dot com

As requested, I attempted the same code with the latest snapshot (php5-200506161630) but __soapCall
continues to exhibit the same buggy behavior as before. As such, the problem doesn't appear to
have been corrected yet. Thank you!

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


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=33366


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


Thread (9 messages)

« previous php.bugs (#185156) next »