httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From William A Rowe Jr <wr...@rowe-clan.net>
Subject Re: mod_http2 1.0.7+ and Drupal (Was: No H2 Window updates!)
Date Mon, 07 Dec 2015 16:13:38 GMT
Within internal_internal_redirect;

  83750   fielding     /*
  83750   fielding      * XXX: hmm.  This is because mod_setenvif and
mod_unique_id really need
  83750   fielding      * to do their thing on internal redirects as well.
Perhaps this is a
  83750   fielding      * misnamed function.
  83750   fielding      */
  83750   fielding     if ((access_status = ap_run_post_read_request(new)))
{
  83750   fielding         ap_die(access_status, new);
  83750   fielding         return NULL;
  83750   fielding     }

That's some ancient mojo there :)


On Mon, Dec 7, 2015 at 6:46 AM, Stefan Eissing <stefan.eissing@greenbytes.de
> wrote:

> I think I just found the problem:
>
> I thought that h2_h2.c, line 678 was no longer needed. But under some,
> still unknown, conditions the post_read_req hook is called twice. That
> makes a table overlay twice which results in all headers_out be present
> twice.
>
> I will remove the filter again before installing it. Someday, I need to
> understand the double invocation of the hook...
>
> //Stefan
>
> > Am 07.12.2015 um 13:38 schrieb Jim Jagielski <jim@jaguNET.com>:
> >
> > Can you retry w DEBUG level, to see if  h2_response_trailers_filter()
> > is the issue?
> >
> >> On Dec 6, 2015, at 6:33 AM, Jan Ehrhardt <phpdev@ehrhardt.nl> wrote:
> >>
> >> Stefan Eissing in gmane.comp.apache.devel (Sun, 6 Dec 2015 09:26:56
> +0100):
> >>> Jan, thanks for tracking this down to the duplicate headers. I will
> look into it.
> >>>
> >>> Just to be sure I got everything right: you tested against the mod_h2
> github versions?
> >>
> >> I used the sources from the ASF repos. This one worked OK:
> >>
> >> svn co -r 1715371
> http://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x/modules/http2
> >>
> >> But this one had the duplicate tokens:
> >>
> >> svn co -r 1716210
> http://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x/modules/http2
> >> --
> >> Jan
> >>
> >
>
>

Mime
View raw message