You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@guacamole.apache.org by ceharris <gi...@git.apache.org> on 2017/12/07 22:56:46 UTC

[GitHub] guacamole-client pull request #215: GUACAMOLE-457: include CAS login URI whe...

GitHub user ceharris opened a pull request:

    https://github.com/apache/guacamole-client/pull/215

    GUACAMOLE-457: include CAS login URI when requesting auth ticket

    When attempting to use the CAS extension with a protocol-compliant CAS server, I discovered that when requesting an authentication ticket the URL used by the extension is missing the `/login` URI path defined in the spec.
    
    This PR adds the necessary additional path segment before appending the service name parameter.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/soulwing/guacamole-client GUACAMOLE-457

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/guacamole-client/pull/215.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #215
    
----
commit 201f07abfdbe9d425368861e0e14638ff0bcfccb
Author: Carl Harris <ce...@vt.edu>
Date:   2017-12-07T22:51:41Z

    GUACAMOLE-457: include CAS login URI when requesting auth ticket

----


---

[GitHub] guacamole-client pull request #215: GUACAMOLE-457: include CAS login URI whe...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/guacamole-client/pull/215


---