Sorry, I found the CVS snapshot extracted on 20021016, but I now found
the problem not fixed. Either my test environment is not good, or my
ability to check all through it is insufficient...

Anyway,

> I don't particularly like the look of this patch... we should fix 304's
> properly...

However, the patch resolved the problem also on the 20021016 snapshot.

> > >
> > > --- sapi/apache2filter/sapi_apache2.c~  Fri Aug 16 07:27:03 2002
> > > +++ sapi/apache2filter/sapi_apache2.c   Mon Oct 14 23:27:26 2002
> > > @@ -558,14 +558,24 @@
> > >         return OK;
> > >  }
> > >
> > > +static int includes_setup(ap_filter_t *f)
> > > +{
> > > +    /* We will ALWAYS set the no_local_copy value to 1 so
> > > +     * that we will not send 304s.
> > > +     */
> > > +    f->r->no_local_copy = 1;
> > > +
> > > +    return OK;
> > > +}
> > > +
> > >  static void php_register_hook(apr_pool_t *p)
> > >  {
> > >         ap_hook_pre_config(php_pre_config, NULL, NULL, APR_HOOK_MIDDLE);
> > >         ap_hook_post_config(php_apache_server_startup, NULL,
> > NULL, APR_HOOK_MIDDLE);
> > >         ap_hook_insert_filter(php_insert_filter, NULL, NULL,
> > APR_HOOK_MIDDLE);
> > >         ap_hook_post_read_request(php_post_read_request, NULL,
> > NULL, APR_HOOK_MIDDLE);
> > > -       ap_register_output_filter("PHP", php_output_filter,
> > NULL, AP_FTYPE_RESOURCE);
> > > -       ap_register_input_filter("PHP", php_input_filter, NULL,
> > AP_FTYPE_RESOURCE);
> > > +       ap_register_output_filter("PHP", php_output_filter,
> > includes_setup, AP_FTYPE_RESOURCE);
> > > +       ap_register_input_filter("PHP", php_input_filter,
> > includes_setup, AP_FTYPE_RESOURCE);
> > >  }
> > >
> > >  AP_MODULE_DECLARE_DATA module php4_module = {
> > >
> >
> >
> > --
> > PHP Development Mailing List <http://www.php.net/>
> > To unsubscribe, visit: http://www.php.net/unsub.php
> >
> >
> 


-- 
TAKAGI Ryo <[EMAIL PROTECTED]>



-- 
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to