https://bz.apache.org/bugzilla/show_bug.cgi?id=57832

Yann Ylavic <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #32735|0                           |1
        is obsolete|                            |

--- Comment #8 from Yann Ylavic <[email protected]> ---
Created attachment 32740
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=32740&action=edit
Proposed patch for 2.4.x on top of r1656259

(In reply to regilero from comment #7)
> I made some tests with the last patch, on top of r1656259 's patch.

Thanks for testing!

> With a big injected response the socket read is
> not empty and is_socket_connected is detecting this fact, but I do not get
> any response (no 502/503/400, just an RST).

Yes, I failed to reset "connected" from 2 to 0 in the previous patch.

> 
> But I think the final solution will have to mix this is_socket_connected and
> a real cleanup of all data read from the socket while processing the first
> response.

Right, we have to also check if data are already in the input filters' stack.
This is what the new attached patch does.

I tested it with both early read and stacked data, but you probably have a more
exhaustive tests suite, can you please give it a (re)try?

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to