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

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

          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