Looks good to me... If it results in problems or issues, we'll fix 'em as the come along ;)
> On Jan 29, 2016, at 8:01 AM, Stefan Eissing <stefan.eiss...@greenbytes.de> > wrote: > > I would like to propose some additions to event that help me get rid of two > ugly hacks in mod_http2: > > 1. Initialization of slave connections > event registers on pre_connection hook and checks if c is a slave > (c->master) and if the connection state is either not there or the same as > master (pointer copy). It then sets up a new cs using the slave connection > pool. > > 2. Initialization of server config > event keeps in its private connection state parts the server configuration > (queues for timeouts and keepalives) that are unique to each server because > of Timeout and KeepAliveTimeout settings. Normally, theses are set on > post_read_request hooks which is fine and well. On master connections > however, these hooks may never get called so the server config remains stuck > on the connections base server. > In the case of SNI however, we know which server this connection is > intended for and should use its Timeouts. This is done by installing an early > protocol_switch hook that sees connections updates and server_rec and > initializes its connection state accordingly. > > Please vote on the attached patch. I do not want to mess with event without > having some moral backup... > > -Stefan > > <event_fix_slaves.patch>