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

Posted to dev@guacamole.apache.org by GitBox <gi...@apache.org> on 2020/07/24 17:27:33 UTC

[GitHub] [guacamole-client] doctorfree commented on pull request #454: GUACAMOLE-793: CAS Provider returns Group - like LDAP Provider

doctorfree commented on pull request #454:
URL: https://github.com/apache/guacamole-client/pull/454#issuecomment-663647931


   Hi, @siacali has asked me to assist with this. After reviewing the changes and discussion here I agree with the approach suggested by @mike-jumper "Perhaps validateTicket() should return the CASAuthenticatedUser instance?". I've created a patch that does this and it is going through some testing. If all works as expected I will resubmit this, rebasing on top of the current master. Does this seem like the right way to go?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org