Skip to content
This repository has been archived by the owner on Mar 16, 2024. It is now read-only.

Fix memory leak in TokenTransport #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Mar 13, 2018

  1. Fix memory leak in TokenTransport

    Close token-demand response before making auth-and-retry.
    
    Unclosed responses lead to unclosed sockets and to memory leaks in long running apps:
    
    ```
    ...
    lrwx------ 1 root root 64 Mar 12 20:16 11 -> socket:[3213896]
    lrwx------ 1 root root 64 Mar 12 20:16 110 -> socket:[3238100]
    lrwx------ 1 root root 64 Mar 12 20:16 111 -> socket:[3238973]
    lrwx------ 1 root root 64 Mar 12 20:16 112 -> socket:[3238317]
    lrwx------ 1 root root 64 Mar 12 20:16 113 -> socket:[3239210]
    lrwx------ 1 root root 64 Mar 12 20:16 114 -> socket:[3239321]
    lrwx------ 1 root root 64 Mar 12 20:16 115 -> socket:[3239503]
    lrwx------ 1 root root 64 Mar 12 20:16 116 -> socket:[3239643]
    lrwx------ 1 root root 64 Mar 12 20:16 117 -> socket:[3238846]
    ...
    ```
    distorhead committed Mar 13, 2018
    Configuration menu
    Copy the full SHA
    dee4b29 View commit details
    Browse the repository at this point in the history