Skip to content

http: Change free sockets behavior to LIFO from FIFO.

Rodrigo Muino Tomonari requested to merge github/fork/rustyconover/fix-https-agent-oldest-sockets into master

Sockets are added to the free list with .push() but they were being removed with .shift(). This meant the sockets where being removed in FIFO order, but this changes it to LIFO. Since older sockets may be closed based due to inactivity on the server it is more likely that a socket that is recently used will be able to successfully process the next request.

  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows

Merge request reports