Spring Security
  1. Spring Security
  2. SEC-925

BasicLookupStrategy - support for schema qualifier

    Details

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

      Description

      The current impl of this class works great - I would like to avoid writing my own custom strategy wherever possible. The JavaDocs for BasicLookupStrategy state that it's not design for extension which I understand.

      It would be great, however, if a property could be set in the bean config that would allow the referenced tables in the SQL to be qualified by a schema. Projects often run into such requirements based on the conventions used by a DBA group. In our particular case, we're using DB2 and the ACL tables are being created in a schema that is not the default schema.

        Issue Links

          Activity

          Hide
          Ross Duncan added a comment -

          Could this be expanded to allow injecting the entire sql query? In some cases the names of the tables involved may not conform to an inhouse policy. Why not allow the entire sql query to be injected as happens in other areas of the security framework?

          Show
          Ross Duncan added a comment - Could this be expanded to allow injecting the entire sql query? In some cases the names of the tables involved may not conform to an inhouse policy. Why not allow the entire sql query to be injected as happens in other areas of the security framework?
          Hide
          Luke Taylor added a comment -

          I guess this should be possible. It would require a select clause, the two where clauses and the "order by".

          Show
          Luke Taylor added a comment - I guess this should be possible. It would require a select clause, the two where clauses and the "order by".
          Hide
          Luke Taylor added a comment -

          All the beans which make use of the schema should be reviewed as part of this issue

          Show
          Luke Taylor added a comment - All the beans which make use of the schema should be reviewed as part of this issue
          Hide
          Luke Taylor added a comment -

          I've added setters for all the SQL which is used by the JDBC implementation, which should solve these issues.

          Show
          Luke Taylor added a comment - I've added setters for all the SQL which is used by the JDBC implementation, which should solve these issues.

            People

            • Assignee:
              Luke Taylor
              Reporter:
              Troy J. Kelley
            • Votes:
              4 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: