Details

    • Type: New Feature New Feature
    • Status: Resolved
    • 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. ApplicationResources-properties.txt
        0.8 kB
        Nick Padilla
      2. LdapCommentAuthenticator.java
        7 kB
        Nick Padilla
      3. roller-properties.txt
        1.0 kB
        Nick Padilla

        Activity

        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!
        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 -

        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 -

        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!
        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 -

        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 -

        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!

          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