Uploaded image for project: 'Single Sign On'
  1. Single Sign On
  2. MGNLSSO-239

Tasks for publishers are only displayed on page refresh using SSO module 3.0.1

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Neutral Neutral
    • saas, 3.1.3
    • 3.0.1
    • None
    • None
    • AdminX 34
    • 3
    • Yes

      Steps to reproduce

      1. Login with an SSO user (not same username like existing MgnlUser in JCR by default, e.g superuser, eric) which belongs to publisher group
      2. Go to pages-app and publish a page
      3. Check that till you don't refresh the browser, the new task doesn't appear

      Expected results

      Once the public task is created, the user can see the indicator within the task app section

      Actual results

      The indicator that a new task exists doesn't appear till the user refresh the browser

      Workaround

      N/A

      Development notes 

      The potential root cause is LocalTaskDispatcherManager#getAllRecipients, the issue happened because the recipients (which is list of username) doesn't contain the current login user to dispatch the new task created event and update the UI (the number in TaskBadge).

      More details, one of the method in SsoUserManager which is called in the method above, throws UnsupportedOperationException because getAllUsers() is not implemented for SSO.

      So, we have to find another way to determine the current login SsoUser is belongs to the group ("publishers" group in this case).

      One concern or limitation is we can't fire events to all users in that group in SSO cases because the users are managed by the Identity provider, I'm not sure whether we should find all users belongs to the group like Mgnl user in JCR. Open to discussion and decide later in grooming cc brenuart mrajkovic 

        Acceptance criteria

          There are no Sub-Tasks for this issue.

              nguyen.phung Nguyen Phung Chi
              ccantalapiedra Carlos Cantalapiedra
              AdminX
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved:
                Work Started:

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - Not Specified
                  Not Specified
                  Logged:
                  Time Spent - 3d 1h
                  3d 1h