Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: ManifoldCF 1.0.1
    • Fix Version/s: ManifoldCF 1.1
    • Component/s: JDBC connector
    • Labels:
      None

      Description

      For scenarios when privileges are based on SQL database with user records and some authorization tokens assigned to them (groups, etc).
      It could help when you have to index number of www sites based on common user-group authorization in database. Also it could help on showcases, presentations and demo versions of "solr-manifold-search client" to show how authorization works.

      (of course I already have this connector written as additional class in default JDBC connector so I could use some of its classes )

        Activity

        Hide
        Karl Wright added a comment -

        I don't see any problem with sharing classes with the JDBC repository connector, provided it is all shipped in the same jar. So that only means it has to be under connectors/jdbc/connector.

        Show
        Karl Wright added a comment - I don't see any problem with sharing classes with the JDBC repository connector, provided it is all shipped in the same jar. So that only means it has to be under connectors/jdbc/connector.
        Hide
        Maciej Lizewski added a comment - - edited

        committed to branch "CONNECTORS-566"

        Show
        Maciej Lizewski added a comment - - edited committed to branch " CONNECTORS-566 "
        Hide
        Karl Wright added a comment -

        I won't have time to review this for a while but if I did, this is what I'd look for.

        • proper integration with the root-level build.xml, so that the JDBC authority registration command is included in connectors*.xml for all models;
        • UI tests for the JDBC authority, which pass

        Once those are in the branch, and the translations too, I will try to get to a quick review, and then you can merge into trunk.

        Show
        Karl Wright added a comment - I won't have time to review this for a while but if I did, this is what I'd look for. proper integration with the root-level build.xml, so that the JDBC authority registration command is included in connectors*.xml for all models; UI tests for the JDBC authority, which pass Once those are in the branch, and the translations too, I will try to get to a quick review, and then you can merge into trunk.
        Hide
        Karl Wright added a comment -

        I committed the translation patch to the branch.

        Show
        Karl Wright added a comment - I committed the translation patch to the branch.
        Hide
        Karl Wright added a comment -

        Looks like you will also need to provide some documentation additions for this one. Let me know if you need help with this.

        Show
        Karl Wright added a comment - Looks like you will also need to provide some documentation additions for this one. Let me know if you need help with this.
        Hide
        Karl Wright added a comment -

        I added the missing parts sufficient for a merge into trunk; will do that shortly.

        Show
        Karl Wright added a comment - I added the missing parts sufficient for a merge into trunk; will do that shortly.
        Hide
        Karl Wright added a comment -

        r1419072 merge from trunk

        Show
        Karl Wright added a comment - r1419072 merge from trunk

          People

          • Assignee:
            Maciej Lizewski
            Reporter:
            Maciej Lizewski
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development