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

Self registration url does not include realm parameter after upgrade from 13.5.1

    Details

    • Sprint:
      2018.16 - Tin, 2019.1 - Chariot, 2019.2 - Glass, 2019.3 - Alphabet, 2019.4 - Coins, 2019.5 - Scissors
    • Needs backport:
      Yes
    • Needs QA verification:
      No
    • Functional tests:
      Yes

      Description

      Bug description

      When performing self registration for a user in a sub-realm (Following an upgrade from 13.5.1), url sent in email link directs user to top-level realm rather than using the subrealm causing registration to fail.

      Working URL for registration on fresh 7.0.0 install is of the format:

       

      [SERVER_URL]/openam/XUI/?realm=[REALM]#register/&token=TOKEN&code=CODE

       

      The URL sent for registration following an upgrade from 13.5.1 is of the format (missing ?realm=[REALM]):

       

      [SERVER_URL]/openam/XUI/#register/&token=[TOKEN]&code=[CODE]

       

      How to reproduce the issue

      1. Deploy AM. Create a subrealm with Email service and User self service
      2. As a user, navigate to the sub-realms login page, click to create an account and then complete the user registration form
      3. Once the registration email is sent successfully, review the link included in the email.

      There is also a Temper functional test which highlights the issue RegisterYourAccountFlowTest

      Work around

      None found, expect that using a realm dns alias could work.

      Code analysis

      Looks to have been introduced as a side-effect of the work for OPENAM-10047 commit 3ef7943a413

      Proposed fix is to add an upgrade step to handle this schema change:

      https://stash.forgerock.org/projects/OPENAM/repos/openam/pull-requests/1978/diff#openam-selfservice/src/main/resources/selfService.xml

       

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                philip.anderson Philip Anderson
                Reporter:
                philip.anderson Philip Anderson
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 0h
                  0h
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 96h
                  96h