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

Elytron: OTP seed as byte array instead of String

XMLWordPrintable

    • 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)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: