Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Message-ID: <473C7F2B.5070409@apache.org>
Date: Thu, 15 Nov 2007 18:17:31 +0100
From: Ruediger Pluem <rpluem@apache.org>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US;
 rv:1.8.1.9) Gecko/20071030 SeaMonkey/1.1.6
MIME-Version: 1.0
To: dev@httpd.apache.org
Subject: Re:
 httpd/sandbox/amsterdam/d/modules/proxy/mod_serf.c?revision=595110&view=markup
References: 
 <99EA83DCDE961346AFA9B5EC33FEC08B05B232@VF-MBX11.internal.vodafone.com>
 <E4324D38-7F7E-48B7-A3CC-D033392415B7@jaguNET.com>
In-Reply-To: <E4324D38-7F7E-48B7-A3CC-D033392415B7@jaguNET.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit


On 11/15/2007 04:26 PM, Jim Jagielski wrote:
> 
> On Nov 15, 2007, at 7:02 AM, Pl�m, R�diger, VF-Group wrote:
> 
>>
>> @@ -567,8 +567,21 @@
>>          return APR_SUCCESS;
>>      }
>>
>> -    if (mode == AP_MODE_EATCRLF || mode == AP_MODE_EXHAUSTIVE ||
>> -        mode == AP_MODE_SPECULATIVE) {
>> +    if (mode == AP_MODE_SPECULATIVE) {
>> +        const char *data;
>> +        apr_size_t len;
>> +        apr_bucket *b;
>> +        serf_bucket_t *sb;
>> +
>> +        ctx->serf_bucket_status = serf_bucket_read(ctx->serf_in_bucket,
>> +                                                   readbytes, &data,
>> &len);
>> +
>> +        sb = serf_bucket_simple_create(data, len, NULL, NULL,
>> ctx->serf_bkt_alloc);
>> +        serf_bucket_aggregate_prepend(ctx->serf_in_bucket, sb);
>>
>> Hm. Don't we need the following lines here?
>>
>>         b = apr_bucket_transient_create(data, len, f->c->bucket_alloc);
>>         APR_BRIGADE_INSERT_TAIL(bb, b);
>>
>> Otherwise I guess we read the data but do not return it :-).
>>
>>
> 
> I don't follow that, but I'm still not quite fully awake yet...

Well, as far as I understand AP_MODE_SPECULATIVE its purpose is to read data and return
it to the caller, but leave the data in the input filter chain such that a following read
(or better a get_brigade) from the input filter chain returns the same data again.
And in the case above we only read the data from ser_in_bucket, but unlike in the AP_MODE_READBYTES
case we do not create a transient bucket with data and add it to the bb brigade. So the
caller gets back an empty brigade with no data.

Regards

R�diger