Uploaded image for project: 'OpenAM'
  1. OpenAM
  2. OPENAM-14677

Cannot use push and OATH registration if attribute does not have a value and user caching is disabled

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Duplicate
    • Affects Version/s: 14.1.1, 14.5.1
    • Fix Version/s: None
    • Component/s: authentication
    • Labels:
    • Sprint:
      AM Sustaining Sprint 62
    • Story Points:
      2
    • Needs backport:
      No
    • Support Ticket IDs:
    • Needs QA verification:
      No
    • Functional tests:
      No
    • Are the reproduction steps defined?:
      Yes and I used the same an in the description

      Description

      Bug description

      During push and OATH registration, if the profile storage attribute does not have a value and user caching is disabled, the user cannot login using that chain.

      How to reproduce the issue

      1. Configure Push Auth following procedure in these docs or OATH following procedure in these docs
      2. Ensure Profile Storage Attribute exists in Data Store > LDAP User Attributes
      3. Disable user caching following procedure in docs
      4. Authenticate to Push Auth or OATH chain
      Expected behaviour
      The same as it is when user caching is enabled. User should be able to login and start push registration
      Current behaviour
      User can't login. Authentication log throws an NPE.

      Work around

      Set a value in the Profile Storage Attribute.

      Code analysis

      org/forgerock/openam/core/rest/devices/UserDevicesDao.java
      for (String profile : set) {
          try {
              devices.add(deviceSerialisation.stringToDeviceProfile(profile));
          } catch (JsonException jve) {
              //RTE, generally indicative that the profile attribute name has changed (still return devices set)
          }
      }
      

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                lawrence.yarham Lawrence Yarham
                Reporter:
                aaron.haskins Aaron Haskins
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: