> My interpretation of fini_received flag means that we have seen a FINI frame 
> or otherwise the client side has indicated that the connection is gone.

There's no FINI frame.. You mean GOAWAY? Anyways, I don't think your 
interpretation of the flag is correct. As you can see, `fini_received` flag is 
set (in the event handler for `HTTP2_SESSION_EVENT_FINI`) regardless of 
client's status nor action (`send_settings_frame` is a good example).

Also, the event handler for `HTTP2_SESSION_EVENT_FINI` is the only place that 
calls `SET_HANDLER(&Http2ConnectionState::state_closed);` This suggests that we 
always need to schedule the event when we close a connection.

So I think the right way to do this is delaying scheduling 
`HTTP2_SESSION_EVENT_FINI`.

[ Full content available at: https://github.com/apache/trafficserver/pull/4097 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to