seeing how people are tagging/building the next .21 beta
do we want this patch applied?
as AS IS using IE and HTTP/1.1 is kinda hosed.
On 17 Jul 2001 11:10:10 -0700, Ian Holsman wrote:
> Victor J. Orlikowski wrote:
>
> > > this seems to fix the problem...
> > > can someone more familiar with brigades/buckets review it
> > >
> > > Index: proxy_http.c
> > > ===================================================================
> > > RCS file: /home/cvspublic/httpd-proxy/module-2.0/proxy_http.c,v
> > > retrieving revision 1.79
> > > diff -u -r1.79 proxy_http.c
> > > --- proxy_http.c 2001/07/16 17:54:38 1.79
> > > +++ proxy_http.c 2001/07/17 16:07:32
> > > @@ -782,18 +782,20 @@
> > >
> > > /* read the body, pass it to the output filters */
> > > while (ap_get_brigade(rp->input_filters, bb, AP_MODE_BLOCKING,
> > > &readbyte
> > > s) == APR_SUCCESS) {
> > > - if (APR_BUCKET_IS_EOS(APR_BRIGADE_LAST(bb))) {
> > > + apr_bucket *b = APR_BRIGADE_LAST(bb);
> > > + if (APR_BUCKET_IS_EOS(b)) {
> > > + APR_BUCKET_REMOVE(b);
> > > e = apr_bucket_flush_create();
> > > APR_BRIGADE_INSERT_TAIL(bb, e);
> > > ap_pass_brigade(r->output_filters, bb);
> >
> >Hum....
> >I may be mistaken, but the end-of-stream bucket still needs to be
> >there. This patch makes certain that there will *never* be an EOS
> >bucket.
> >
> I'm still seeing a EOS bucket being processed
> [Tue Jul 17 09:07:20 2001] [error] [client 127.0.0.1] EOS Bucket in
> Chunk_filter /test/main.shtml (created at
> c:\src\apache\httpd-2.0\server\protocol.c 736)
>
> >
> >
> >Feel free to correct me on this.
> >
> >Victor
> >
>
>
--
Ian Holsman [EMAIL PROTECTED]
Performance Measurement & Analysis
CNET Networks - (415) 364-8608
