HttpComponents HttpClient
  1. HttpComponents HttpClient
  2. HTTPCLIENT-1143

CachingHttpClient leaks connections with stale-if-error

    Details

      Description

      If you are using the "stale-if-error" Cache-control header and CachingHttpClient decides to use a stale cached response it does not clean up the existing backend response.

      This bug causes connections to leak from the connection pool each time the stale-if-error flow is executed.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        2h 22m 1 Jon Moore 07/Nov/11 21:48
        Resolved Resolved Closed Closed
        697d 21h 52m 1 Oleg Kalnichevski 05/Oct/13 20:41
        Oleg Kalnichevski made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Jon Moore made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Jon Moore added a comment -

        Patch applied and backported to 4.1.x release branch.

        Show
        Jon Moore added a comment - Patch applied and backported to 4.1.x release branch.
        Hide
        Jon Moore added a comment -

        Patch checked in to trunk with slight modifications, including unit test to reproduce. Thanks, James! I'll work on getting this ported to the release branches next.

        Show
        Jon Moore added a comment - Patch checked in to trunk with slight modifications, including unit test to reproduce. Thanks, James! I'll work on getting this ported to the release branches next.
        Jon Moore made changes -
        Fix Version/s 4.1.3 [ 12317840 ]
        Fix Version/s 4.2 Alpha2 [ 12318879 ]
        Fix Version/s Future [ 12312298 ]
        Hide
        Jon Moore added a comment -

        Hi James, thanks for finding this. I'm actually working on a unit test to reproduce, although I agree you've found a bug and your patch should address it. Just dotting all the i's, so to speak.

        Show
        Jon Moore added a comment - Hi James, thanks for finding this. I'm actually working on a unit test to reproduce, although I agree you've found a bug and your patch should address it. Just dotting all the i's, so to speak.
        Jon Moore made changes -
        Assignee Jon Moore [ jonm ]
        James Miller made changes -
        Field Original Value New Value
        Attachment httpclient-1143.patch [ 12502786 ]
        Hide
        James Miller added a comment -

        patch for stale-if-error connection leak

        Show
        James Miller added a comment - patch for stale-if-error connection leak
        James Miller created issue -

          People

          • Assignee:
            Jon Moore
            Reporter:
            James Miller
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development