Karaf
  1. Karaf
  2. KARAF-902

Jaas: Properties backing engine doesn't remove users

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.2.3
    • Fix Version/s: 2.2.4, 3.0.0
    • Component/s: None
    • Labels:
      None

      Description

      users.save() should be called, after removing user from properties

      1. jaas.patch
        0.8 kB
        Dejan Bosanac

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        3h 33m 1 Jamie goodyear 27/Sep/11 19:55
        Resolved Resolved Closed Closed
        118d 18h 58m 1 Jamie goodyear 24/Jan/12 13:54
        Jamie goodyear made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Jamie goodyear made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 3.0.0 [ 12316040 ]
        Resolution Fixed [ 1 ]
        Hide
        Jamie goodyear added a comment -

        Thanks Dejan Bosanac for the patch.

        Show
        Jamie goodyear added a comment - Thanks Dejan Bosanac for the patch.
        Hide
        Jamie goodyear added a comment -

        Karaf Trunk:

        $ svn ci
        Sending jaas/modules/src/main/java/org/apache/karaf/jaas/modules/properties/PropertiesBackingEngine.java
        Transmitting file data .
        Committed revision 1176522.

        Show
        Jamie goodyear added a comment - Karaf Trunk: $ svn ci Sending jaas/modules/src/main/java/org/apache/karaf/jaas/modules/properties/PropertiesBackingEngine.java Transmitting file data . Committed revision 1176522.
        Hide
        Jamie goodyear added a comment -

        Ok, the capitalizes "LOGGER" was required for trunk, using the original patch for that line.

        Show
        Jamie goodyear added a comment - Ok, the capitalizes "LOGGER" was required for trunk, using the original patch for that line.
        Hide
        Jamie goodyear added a comment -

        Karaf 2.2.x branch:

        $ svn ci
        Sending jaas/modules/src/main/java/org/apache/karaf/jaas/modules/properties/PropertiesBackingEngine.java
        Transmitting file data .
        Committed revision 1176501.

        Show
        Jamie goodyear added a comment - Karaf 2.2.x branch: $ svn ci Sending jaas/modules/src/main/java/org/apache/karaf/jaas/modules/properties/PropertiesBackingEngine.java Transmitting file data . Committed revision 1176501.
        Hide
        Jamie goodyear added a comment -

        Thanks Dejan for the patch, I'm making a small modification to make "LOGGER" lowercase, otherwise things look good

        Show
        Jamie goodyear added a comment - Thanks Dejan for the patch, I'm making a small modification to make "LOGGER" lowercase, otherwise things look good
        Jamie goodyear made changes -
        Fix Version/s 2.2.4 [ 12317342 ]
        Jamie goodyear made changes -
        Assignee Jamie goodyear [ jgoodyear ]
        Dejan Bosanac made changes -
        Field Original Value New Value
        Attachment jaas.patch [ 12496701 ]
        Dejan Bosanac created issue -

          People

          • Assignee:
            Jamie goodyear
            Reporter:
            Dejan Bosanac
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development