Uploaded image for project: 'Guacamole'
  1. Guacamole
  2. GUACAMOLE-457

CAS authentication provider omits login URI

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.9.13-incubating
    • Fix Version/s: 1.0.0
    • Component/s: None
    • Labels:
      None

      Description

      According to the CAS 2.0 protocol specification (https://apereo.github.io/cas/5.1.x/protocol/CAS-Protocol-V2-Specification.html) the URI for obtaining a ticket for an unauthenticated user is derived from the base URI by appending `/login`. In the current source for CAS authentication provider, the URI that is used for this purpose is the base URI (i.e. whatever URL is configured as the value for the `cas-authorization-endpoint` property). This prevents successful authentication using the provider with a protocol-compliant CAS server.

      Attempting to use the login URL as the value of the `cas-authorization-endpoint` property as a workaround subsequently fails when the validation URI is appended to the URL; this results in a URL that ends with the path `/login/proxyValidate` which is incorrect and violates the CAS protocol.

      It seems that the solution could be as simple as appending `/login` to the authorization endpoint URI in the AuthorizationServiceProvider, when the CASTicketField object is constructed. Indeed a simple patch using this approach was tested and confirmed to work properly with a protocol-compliant CAS server implementation.

      However, this approach allows some protocol specifics to leak into the AuthorizationServiceProvider which otherwise relies on abstractions that wrap classes of the Apereo CAS client. It does not appear that there is an appropriate class/method provided by the CAS client library to construct the login URL from the base URL. Perhaps simply adding a method to the extension's TicketValidationService for this purpose would be reasonable.

        Attachments

          Activity

            People

            • Assignee:
              ceharris Carl Harris
              Reporter:
              ceharris Carl Harris
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: