Uploaded image for project: 'WildFly Elytron'
  1. WildFly Elytron
  2. ELY-1288

Elytron: OTP seed as byte array instead of String


    • Icon: Bug Bug
    • Resolution: Duplicate
    • Icon: Major Major
    • 1.1.0.CR3
    • None
    • None
    • None

      The org.wildfly.security.password.interfaces.OneTimePassword interface contains getSeed() method which is of type byte[]. The more proper type seems to be a String (or char[]).

      The OneTimePassword interface type description says:

      A one-time password, used by the OTP SASL mechanism.

      The OTP RFC 2289 says

         The seed MUST consist of purely alphanumeric characters and MUST be
         of one to 16 characters in length. The seed is a string of characters
         that MUST not contain any blanks and SHOULD consist of strictly
         alphanumeric characters from the ISO-646 Invariant Code Set.  The
         seed MUST be case insensitive and MUST be internally converted to
         lower case before it is processed.

      Suggested fix:
      Change the getSeed() method type to String.

            yborgess1@redhat.com Yeray Borges Santana
            josef.cacek@gmail.com Josef Cacek (Inactive)
            0 Vote for this issue
            1 Start watching this issue