Added link to github with a simple fix for this.

Thanks for reporting this and providing a solution.

I didn't test, but I know about this issue with SOAP server. The patch seems good to me, Mike, you can add to the next sprint and keep it assigned to me.

Thank you Adam.

Hello,
I do not like this change much because it only hides the fact that some code is doing something wrong. I looked at the code and it seems that the headers are sent only once in all cases. Could you get me a stack trace of the first and second sending of headers?

Petr

Thanks for the comment Petr, it got me to look in the right place.

I believe the issue is that the soap server is outputting content before the headers. Naturally as soon as you dump out content, headers are in effect sent.
In webservice/lib.php's run(), $response = $this->zend_server->handle(); is called before $this->send_headers();. handle() will call dump(), which just echoes out data.

I've changed my commit to what makes more sense given this. It moves the send_headers call before any output is possibly sent.

makes sense, thanks!

To integrators: please cherry pick to all 2.x branches

(fixing repository and branch)

Integrated, thanks! (backported to 20 and 21 stable)

Works Great
Thanks for fixing this Adam and Petr.

Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.