Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-14622

In LDAP plugin, #checkPassword leaves an orphaned LDAP context which causes a memory leak

XMLWordPrintable

    • Icon: Bug Report Bug Report
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 7.18.0, 7.18.0-alpha1
    • None
    • engine
    • None

      Environment (Required on creation):

      Any distribution, LDAP identity plugin enabled.

      Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket):

      The checkPassword method leaves an orphaned LDAP context. If the resource is not closed properly, it can lead to memory leaks.

      Steps to reproduce (Required on creation):

      1. Enable the LDAP identity plugin in any distribution.
      2. Stress test user login (the checkPassword method), e.g.:
        1. Enable authentication for the REST API.
        2. Repeatedly call any method that requires authentication from a stress test tool.

      Observed Behavior (Required on creation):

      A memory leak.

      Expected behavior (Required on creation):

      No memory leak.

      Root Cause (Required on prioritization):

      Solution Ideas (Optional):

      Close the context in a finally block, see this PR

      Hints (optional):

        This is the controller panel for Smart Panels app

              tassilo.weidner Tassilo Weidner
              daniel.ewing Daniel Ewing
              Tassilo Weidner Tassilo Weidner
              Tassilo Weidner Tassilo Weidner
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Created:
                Updated:
                Resolved: