XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • TBD
    • Resolution: Done
    • 2.2.0, 2.7.0, 2.8.0, 2.9.0
    • 2.9.1, 2.10.0
    • None
    • CP: sprint 128
    • 1
    • Core: Platform

    Description

      mod-notify leaks a socket when performing POST /notify ..

      Can be demonstrated in a unit test by repeated calls to POST /notify.

      It is also clear from code inspection that a WebClient is constructed for calls to postNotify -> getOkapiModulesClient - > OkapiModulesClientImpl.

      OkapiModulesClientImpl never closes webClient (not that it should, but better than creating a pool every time)

      Instead OkapiModulesClientImpl should create only one WebClient per Vert.x instance.

      Repeating test SendNotifyTest.testValidRequest is a way to reveal that number of sockets. Unfortunately, not so easy to
      make a test for number of sockets in use.

      TestRail: Results

        Attachments

          Issue Links

            Activity

              People

                adam Adam Dickmeiss (Inactive)
                adam Adam Dickmeiss (Inactive)
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  TestRail: Runs

                    TestRail: Cases