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

Add new configuration postLogoutRedirectUri for SSO logout

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Neutral
    • 2.0.1
    • None
    • None
    • Yes
    • Global Maintenance 2
    • 3

    Description

      Steps to reproduce

      1. Install SSO module
      2. Configure against Microsoft Azure
      3. Login to the system
      4. Logout

      Expected results

      post_logout_redirect_uri parameter is successfully recognized by Azure as correct reply url

      Actual results

      post_logout_redirect_uri parameter is wrong (not the one configured as reply url but the request.getRequestURL() value)

      Workaround

      N/A

      Development notes

      The value added is based on request.getRequestURL(). This is not correct in many cases.
      E.g. we have tomcat running on http and nginx running SSL. tomcat doesn't know about ssl and identifies request.getRequestURL() as http url.
      post_logout_redirect_uri value should be based on whatever is configured via "authenticationService.path"

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Bug DoR
                    Task DoD

                    Time Tracking

                      Estimated:
                      Original Estimate - Not Specified
                      Not Specified
                      Remaining:
                      Remaining Estimate - Not Specified
                      Not Specified
                      Logged:
                      Time Spent - 2d
                      2d