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

Demo user creation is based on whether a userCfg is specified, rather than when it's set to embedded

    Details

    • Sprint:
      Team Maxwell - 2017.15
    • Needs backport:
      No
    • Support Ticket IDs:
    • Needs QA verification:
      No
    • Functional tests:
      No
    • Are the reproduction steps defined?:
      Yes but I used my own steps. (If so, please add them in a new comment)

      Description

      Bug description

      Demo user is supposed to be created when the embedded user repo is used. However, recent changes to Amster have exposed a flaw in the logic which actually bases the choice on creating the Demo user on whether the user repo is not specified rather than set to embedded.

      How to reproduce the issue

      1. Install OpenAM using amster install-openam (it defaults to specifying embedded)
      2. View Subjects in OpenAM
      3. Note lack of Demo user
      Expected behaviour
      Demo User exists
      
      Current behaviour
      Demo user does not exist
      

      Code analysis

      com.sun.identity.setup.AMSetupServlet.java
      if ((userRepo == null) || userRepo.isEmpty()) {
          createDemoUser();
      }
      

      Should be changed such that we check isDSEmbedded()

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                emma.rumsey Emma Rumsey
                Reporter:
                david.luna@forgerock.com David Luna
              • Votes:
                0 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: