Spring Security
  1. Spring Security
  2. SEC-1172

AclEntryAfterInvocationProvider has hard coded config attribute of AFTER_ACL_READ, should provide a constructor allowing client code to provide alternate config string

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0.0 M2
    • Component/s: ACLs
    • Labels:
      None

      Description

      The constructor for AclEntryAfterInvocationProvider hard codes the config attribute as AFTER_ACL_READ.

      There should probably be an additional constructor allowing client code to supply a value.

        Activity

        Hide
        Luke Taylor added a comment -

        You can modify the default using the property "processConfigAttribute". So I don't think this is really necessary. This class will also be largely deprecated in favour of an expression-based approach in Spring Security 3.

        Show
        Luke Taylor added a comment - You can modify the default using the property "processConfigAttribute". So I don't think this is really necessary. This class will also be largely deprecated in favour of an expression-based approach in Spring Security 3.
        Hide
        Taylor Mathewson added a comment -

        The setter is protected.

        Show
        Taylor Mathewson added a comment - The setter is protected.
        Hide
        Luke Taylor added a comment -

        Ah yes, well spotted . I've added an extra constructor as per the original request.

        Show
        Luke Taylor added a comment - Ah yes, well spotted . I've added an extra constructor as per the original request.

          People

          • Assignee:
            Luke Taylor
            Reporter:
            Taylor Mathewson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: