Content-Length 0 in HTTP plugin responses

Unable to reproduce on trunk on Ubuntu raring, though its got some other weird effect:

$ telnet localhost 8999
Trying ::1...
Connected to localhost.localdomain.
Escape character is '^]'.
POST /reverse HTTP/1.1
Content-Length: 12

Hello world!
!dlrow olleH OK
X-Gearman-Job-Handle: H:clint-MacBookPro:5
X-Gearman-Command: GEARMAN_COMMAND_WORK_COMPLETE
Content-Length: 12
Server: Gearman/1.1.6

!dlrow olleH

Notice that instead of the appropriate HTTP response code, '200 OK', its the result followed by OK.

My version is 1.1.5 where i can download the trunk version, i cannot find the SVN.

At protocol.cc:217 "content.size()" allways look 0.

I am testing at CentOS release 6.3 (Final) and the same issue with 1.1.5.

Thanks for your fast reply

Its a bit weird i compiled the trunk version too, and i get the same result:
$ telnet localhost 1111
Trying ::1...
Connected to localhost.
Escape character is '^]'.
POST /reverse HTTP/1.1
Content-Length: 12

Hello world!
HTTP/1.0 200 OK
X-Gearman-Job-Handle: H:mc-air:21
X-Gearman-Command: GEARMAN_COMMAND_WORK_COMPLETE
Content-Length: 0
Server: Gearman/1.1.6

!dlrow olleH

This is with the code from: "bzr branch lp:gearmand" compiled with:
./configure --disable-libdrizzle --disable-hiredis --disable-libmemcached --disable-libmysql --disable-libpq --disable-libsqlite3 --disable-option-checking --with-sqlite3=no --with-mysql=no

And the worker code is:
<?php

$gmworker= new GearmanWorker();
$gmworker->addServer('localhost', 4730);
$gmworker->addFunction('reverse', function($job) {
    return strrev($job->workload());
});

print "Waiting for job...\n";
while( $gmworker->work() ) {
    if ($gmworker->returnCode() != GEARMAN_SUCCESS) {
        echo "return_code: " . $gmworker->returnCode() . "\n";
        break;
    }
}

Odd... length is used to determine output of the body,... so if it was zero then nothing should print in the body.

Yes then is a issue right? Because gearmand is returning this. :/

bug fixing in gearman 1.0.6 proposed in https://code.launchpad.net/~1-infe-w/gearmand/1.0/+merge/244981

The patch looks to be stuck in gearmand's current vortex of merge proposals. Hopefully we can un-stick that.

gearman 1.1.12 bug fix proposal
https://code.launchpad.net/~1-infe-w/gearmand/gearmand/+merge/271083

palik, can we move this over to a github PR?

This is already done in git commit f12969e

https://github.com/gearman/gearmand/pull/2/commits/f12969ea7ab028baad17753b22a1e37435464b74

PR 2 solves the issue for POST requests but not for GET. I'll improve libgearman-server/plugins/protocol/http/protocol.cc