Uploaded image for project: 'Jackrabbit Oak'
  1. Jackrabbit Oak
  2. OAK-3201

Use static references in SecurityProviderImpl for composite services

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.22, 1.2.7, 1.3.7, 1.4
    • Component/s: core
    • Labels:
      None

      Description

      SecurityProviderImpl has dynamic references to many other services, like RestrictionProvider, that represent the configuration of this component.

      Being these services dynamic, the OSGi runtime has no clear dependency relationship between the SecurityProviderImpl and the required services. Thus, it may happen that an instance of SecurityProviderImpl is published before the services it requires are started, creating a window where the SecurityProviderimpl is operating differently from the way it's configured.

      I suggest to turn the dynamic references in SecurityProviderImpl to static ones to improve the consistency of the implementation.

        Attachments

        1. OAK-3201-09.patch
          57 kB
          Francesco Mari
        2. OAK-3201-08.patch
          68 kB
          Francesco Mari
        3. OAK-3201-07.patch
          79 kB
          Francesco Mari
        4. OAK-3201-06.patch
          63 kB
          Francesco Mari
        5. OAK-3201-05.patch
          48 kB
          Francesco Mari
        6. OAK-3201-04.patch
          15 kB
          Francesco Mari
        7. OAK-3201-03.patch
          23 kB
          Francesco Mari
        8. OAK-3201-02.patch
          12 kB
          Francesco Mari
        9. OAK-3201-01.patch
          5 kB
          Francesco Mari
        10. mbean-test.log
          32 kB
          Chetan Mehrotra

          Issue Links

            Activity

              People

              • Assignee:
                frm Francesco Mari
                Reporter:
                frm Francesco Mari
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: