Uploaded image for project: 'Jackrabbit Content Repository'
  1. Jackrabbit Content Repository
  2. JCR-3495

Unregister from PrivilegeRegistry and NodeTypeRegistry on Session.logout()

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 2.4.5, 2.6.1, 2.7
    • nodetype, security
    • None

    Description

      PrivilegeRegistry and NodeTypeRegistry contains weak references to a session even if the session is explicitly logged out. While PrivilegeRegistry is designed that way and currently does not allow a client to unregister a listener, the NodeTypeRegistry does have a method to unregister, but it is not used by NodeTypeManagerImpl.

      This puts extra load on the finalizer and should be avoided.

      Attachments

        1. JCR-3495.patch
          4 kB
          Marcel Reutegger

        Activity

          Proposed changes.

          mreutegg Marcel Reutegger added a comment - Proposed changes.

          Committed patch in revision: 1445122.

          mreutegg Marcel Reutegger added a comment - Committed patch in revision: 1445122.
          stillalex Alex Deparvu added a comment -

          backported to 2.6 with rev 1479817.

          stillalex Alex Deparvu added a comment - backported to 2.6 with rev 1479817.
          stillalex Alex Deparvu added a comment -

          backported to 2.4 with rev 1524732.

          stillalex Alex Deparvu added a comment - backported to 2.4 with rev 1524732.

          People

            Unassigned Unassigned
            mreutegg Marcel Reutegger
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: