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

ServiceConfigManagerImpl does not retain order of notification events.

    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: None
    • Labels:
      None
    • Target Version/s:
    • Needs backport:
      No
    • Needs QA verification:
      No
    • Functional tests:
      No
    • Are the reproduction steps defined?:
      No (add reasons in the comment)

      Description

      Bug description

      ServiceConfigManagerImpl gets informed of events, and later is required to notify other elements of these events. However, the order in which the events come out is not guaranteed to match the order they came in.

      Code analysis

      ServiceConfigManagerImpl accepts events and stores them in a HashMap, which it later places into a HashSet before iterating over the set to trigger the appropriate ServiceListeners.

      By using a LinkedHashMap and pulling the values out in order into a LinkedSet, the order of events will instead be retained.

        Attachments

          Activity

            People

            • Assignee:
              david.luna@forgerock.com David Luna
              Reporter:
              david.luna@forgerock.com David Luna
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: