Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.1
    • Component/s: None
    • Level of effort:
      just use my implementation.

      Description

      Enable restriction so that only LDAP authenticated users can add comments. Please see attached files for implementation. I didn't do any translations for the ApplicationResources.properties change I made, just fyi. Just enable it the same way you would the MathCommentAuthenticator, plus a few settings so we can find the ldap server.

      1. roller-properties.txt
        1.0 kB
        Nick Padilla
      2. LdapCommentAuthenticator.java
        7 kB
        Nick Padilla
      3. ApplicationResources-properties.txt
        0.8 kB
        Nick Padilla

        Activity

        Nick Padilla created issue -
        Hide
        Nick Padilla added a comment -

        Files to enable this feature. Hope i did the diff files right, first time for me!

        Show
        Nick Padilla added a comment - Files to enable this feature. Hope i did the diff files right, first time for me!
        Nick Padilla made changes -
        Field Original Value New Value
        Attachment ApplicationResources-properties.txt [ 12511792 ]
        Attachment LdapCommentAuthenticator.java [ 12511793 ]
        Attachment roller-properties.txt [ 12511794 ]
        Hide
        Nick Padilla added a comment -

        Any idea when this new feature will be added to roller? Just wanted to check in.

        Show
        Nick Padilla added a comment - Any idea when this new feature will be added to roller? Just wanted to check in.
        Hide
        Nick Padilla added a comment -

        Just wanted to check in on this issue, is it possible to get this in by the next release? Let me know if there are any questions or concerns.

        Thanks!

        Show
        Nick Padilla added a comment - Just wanted to check in on this issue, is it possible to get this in by the next release? Let me know if there are any questions or concerns. Thanks!
        Hide
        Nick Padilla added a comment -

        Hello All!

        I hope everyone is doing well! Haven't seen any movement on this project for a few weeks! I wanted to check in and see the viability of adding this feature to the next release. Let me know if anyone has any questions.

        Thanks!

        Show
        Nick Padilla added a comment - Hello All! I hope everyone is doing well! Haven't seen any movement on this project for a few weeks! I wanted to check in and see the viability of adding this feature to the next release. Let me know if anyone has any questions. Thanks!
        Dave Johnson (Inactive) made changes -
        Assignee Roller Unassigned [ roller_unassigned ] Dave Johnson [ snoopdave ]
        Hide
        Dave Johnson (Inactive) added a comment -

        Hi Nick, for some reason I forgot all about this patch when I integrated your previous work. I hope to get it into Roller trunk this weekend.

        Show
        Dave Johnson (Inactive) added a comment - Hi Nick, for some reason I forgot all about this patch when I integrated your previous work. I hope to get it into Roller trunk this weekend.
        Hide
        Nick Padilla added a comment -

        Hey Dave! Thanks a million! Let me know if you need anything.

        Show
        Nick Padilla added a comment - Hey Dave! Thanks a million! Let me know if you need anything.
        Hide
        Dave Johnson (Inactive) added a comment -

        Patch committed. Thanks for your contribution!

        Show
        Dave Johnson (Inactive) added a comment - Patch committed. Thanks for your contribution!
        Dave Johnson (Inactive) made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Nick Padilla made changes -
        Fix Version/s 5.1 [ 12317621 ]
        Glen Mazza made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        61d 9h 26m 1 Dave Johnson (Inactive) 26/Mar/12 13:17
        Resolved Resolved Closed Closed
        882d 5h 39m 1 Glen Mazza 25/Aug/14 18:56

          People

          • Assignee:
            Dave Johnson (Inactive)
            Reporter:
            Nick Padilla
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 1h
              1h
              Remaining:
              Remaining Estimate - 1h
              1h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development