incorrect content-length on json widget download (off by one) about homer-api

[davehorton]

This happens not only on file downloads, but also GET requests to api/v1/search/data -- again, only if the amount of data is large enough. If I filter only one or a few records, they display but if I bring back 1000 then the content length is off and I get a JSON parse error as a result

[davehorton]

I'm noticing that the latest checked-in homer-api has this:

    public function setContentLength($pMessage)
    {
	//Fix for MAC Safary and Chrome + 2 for response
        if (php_sapi_name() !== 'cli' )
            header('Content-Length: '.strlen($pMessage)+2);
    }

while my running container has this:

    public function setContentLength($pMessage)
    {
        if (php_sapi_name() !== 'cli' )
            header('Content-Length: '.strlen($pMessage));
    }

This looks like it could be the cause, I will investigate updating my image

[davehorton]

ok so it looks like it was fixed in this commit:
ec17e43
but the homer-complete image does not include it:
https://github.com/sipcapture/homer-docker/blob/master/webapp/Dockerfile#L20

I can deploy that fix and build my own image, but wondering if there is plans to release a new homer-complete image at some point with this fix?

If my diagnosis is faulty please let me know

[lmangani]

Thanks for the update @davehorton - I guess all we need is retrigger the Docker builders. I'm trying so now, will confirm once ready to retest, please standby

[adubovikov]

http://install.sipcapture.io/