Uploaded image for project: 'OpenAM'
  1. OpenAM
  2. OPENAM-12262

CachedSMSEntry should only deregister its listener upon invalidation

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 6.0.0
    • Fix Version/s: 6.0.0, 5.5.2
    • Component/s: sms
    • Labels:
      None
    • Needs backport:
      Yes
    • Needs QA verification:
      No
    • Functional tests:
      Yes
    • Are the reproduction steps defined?:
      Yes and I used the same an in the description

      Description

      Bug description

      At the moment CachedSMSEntry deregisters its listener from SMSEventListenerManager as soon as there are no more serviceObjects registered with it, even though the instance is still valid. The problem is that the listener is never re-registered when adding the first listener again, hence a CachedSMSEntry that remains valid for a long interval may not receive notifications for the most part of its lifetime.

      How to reproduce the issue

      Execute functional tests like TestAuthModuleInChain and BlacklistingAuthSessions.

      Expected behaviour

      The listener isn't lost when serviceObjects becomes empty

      Current behaviour

      Listeners are lost.

        Attachments

          Activity

            People

            • Assignee:
              peter.major Peter Major [X] (Inactive)
              Reporter:
              peter.major Peter Major [X] (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: