Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: ssl socket leak when keepalive is used #5713

Closed
wants to merge 1 commit into from
Closed

fix: ssl socket leak when keepalive is used #5713

wants to merge 1 commit into from

Commits on Mar 15, 2016

  1. fix: ssl socket leak when keepalive is used

    SSL sockets leak whenever keep alive is enabled, ca option is set in the
    global agent, and requests are sent without the ca property.
    In the following case at Agent.prototype.createSocket a socket will be
    created with a hashtag name that includes data from the global agents’
    ca property.
    On subsequent requests at Agent.prototype.addRequest we do not find the
    free socket, because the hashtag name generated there does not take into
    account the global agents’ ca property, thus creating a new socket and
    leaving the first socket to timeout.
    -closes: #5699
    alexpenev-s committed Mar 15, 2016
    Configuration menu
    Copy the full SHA
    d67d65b View commit details
    Browse the repository at this point in the history