Derby
  1. Derby
  2. DERBY-5693

BUILTIN should say passwords are hashed not encrypted

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 10.9.1.0
    • Fix Version/s: 10.9.1.0
    • Component/s: Services
    • Labels:
      None

      Description

      Many places in the BUILTIN authentication code it is said that passwords are encrypted before they are stored in the database. It would be more accurate to say that the passwords are hashed.

      1. d5693-1a.diff
        10 kB
        Knut Anders Hatlen

        Activity

        Knut Anders Hatlen created issue -
        Knut Anders Hatlen made changes -
        Field Original Value New Value
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Knut Anders Hatlen added a comment -

        Attaching a patch that changes "encrypt" to "hash" in the code comments in AuthenticationServiceBase and BasicAuthenticationServiceImpl. It also renames three methods in those classes to make it clearer that they don't encrypt the passwords, but rather hash them.

        Show
        Knut Anders Hatlen added a comment - Attaching a patch that changes "encrypt" to "hash" in the code comments in AuthenticationServiceBase and BasicAuthenticationServiceImpl. It also renames three methods in those classes to make it clearer that they don't encrypt the passwords, but rather hash them.
        Knut Anders Hatlen made changes -
        Attachment d5693-1a.diff [ 12523173 ]
        Hide
        Knut Anders Hatlen added a comment -

        All the regression tests ran cleanly with the patch.

        Show
        Knut Anders Hatlen added a comment - All the regression tests ran cleanly with the patch.
        Hide
        Knut Anders Hatlen added a comment -

        Committed revision 1327910.

        Show
        Knut Anders Hatlen added a comment - Committed revision 1327910.
        Knut Anders Hatlen made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Issue & fix info Patch Available [ 10102 ]
        Fix Version/s 10.9.0.0 [ 12316344 ]
        Resolution Fixed [ 1 ]
        Knut Anders Hatlen made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Gavin made changes -
        Workflow jira [ 12662959 ] Default workflow, editable Closed status [ 12797040 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        1m 56s 1 Knut Anders Hatlen 18/Apr/12 12:04
        In Progress In Progress Resolved Resolved
        1d 55m 1 Knut Anders Hatlen 19/Apr/12 13:00
        Resolved Resolved Closed Closed
        96d 5h 28m 1 Knut Anders Hatlen 24/Jul/12 18:28

          People

          • Assignee:
            Knut Anders Hatlen
            Reporter:
            Knut Anders Hatlen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development