SourceForge has been redesigned. Learn more.
Close

#5160 http package, KeepAlive, reset tokens closed shared connects

obsolete: 8.5.13
open
5
2013-01-15
2013-01-07
No

http package does not resets token state by KeepAlive. So it will be not reconnected and still waiting...
Patch for http 2.7.10 attached.
Unfortunately, I have no test case for this. But it is understandable by patch.

Discussion

  • Serg G. Brester

    Serg G. Brester - 2013-01-07

    reset keepalive connection patch

     
  • Jan Nijtmans

    Jan Nijtmans - 2013-01-10
    • assigned_to: patthoyts --> dkf
     
  • Jan Nijtmans

    Jan Nijtmans - 2013-01-10

    Donal, I think you know the http package better than I do. Is this the right way?

     
  • Donal K. Fellows

    Just a note to say that I've seen that this is assigned to me, but it will take a few days to evaluate because of work pressures.

     
  • Donal K. Fellows

    • milestone: --> obsolete: 8.5.13
     
  • Donal K. Fellows

    Understanding what the 'socketmap' variable actually maps between is rather harder than puzzling out what the patch does. In particular, I'm trying to figure out if the 'sockettokenmap' should be used as the inverse of 'socketmap', as if it should then there's code that should be simplified.

     
  • Jan Nijtmans

    Jan Nijtmans - 2013-01-24

    Proposed fix committed to "bug-3599789" branch.