Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      Moving classes to auth package and converting some classes to interfaces with base class. This will make your docs outdated. Also please get stuff into the xdocs too. Will be nice to put this on the site.

      Thanks!

      1. authenticator2.patch
        26 kB
        Endi S. Dewata
      2. authenticator-javadoc.zip
        43 kB
        Endi S. Dewata

        Activity

        Hide
        Alex Karasulu added a comment -

        Also there is no javadocs in the code. Please make a habit of javadoc documenting your methods. Note that this is even more critical with SPIs that you have contributed. People need to be able to understand how to implement the methods of those SPIs.

        Show
        Alex Karasulu added a comment - Also there is no javadocs in the code. Please make a habit of javadoc documenting your methods. Note that this is even more critical with SPIs that you have contributed. People need to be able to understand how to implement the methods of those SPIs.
        Hide
        Endi S. Dewata added a comment -

        Here's a patch (authenticator2.patch) containing the xdocs for Authenticator SPI. I also cleaned up the interface a little bit. I'm also attaching a zip file (authenticator-javadoc.zip) containing the Javadoc (not sure where to put it).

        Show
        Endi S. Dewata added a comment - Here's a patch (authenticator2.patch) containing the xdocs for Authenticator SPI. I also cleaned up the interface a little bit. I'm also attaching a zip file (authenticator-javadoc.zip) containing the Javadoc (not sure where to put it).
        Hide
        Alex Karasulu added a comment -
        Show
        Alex Karasulu added a comment - Committed revision 159239. http://svn.apache.org/viewcvs.cgi?rev=159239&view=rev

          People

          • Assignee:
            Alex Karasulu
            Reporter:
            Alex Karasulu
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development