[GitHub] incubator-guacamole-client pull request #183: GUACAMOLE-362: Add support for...

[mike-jumper <gi...@git.apache.org>]

Github user mike-jumper commented on a diff in the pull request:

    https://github.com/apache/incubator-guacamole-client/pull/183#discussion_r139978510
  
    --- Diff: extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/ConfigurationService.java ---
    @@ -68,4 +70,20 @@ public String getRedirectURI() throws GuacamoleException {
             return environment.getRequiredProperty(CASGuacamoleProperties.CAS_REDIRECT_URI);
         }
     
    +    /**
    +     * Returns the path to the file that contains the private key
    +     * used to decrypt the credential that is sent encrypted by CAS,
    +     * or null if no key is defined.
    +     *
    +     * @return
    +     *     The path to the private key to decrypt the ClearPass
    +     *     credential returned by CAS.
    +     *
    +     * @throws GuacamoleException
    +     *     If guacamole.properties cannot be parsed.
    +     */
    +    public Cipher getClearpassCipher() throws GuacamoleException {
    +        return environment.getProperty(CASGuacamoleProperties.CAS_CLEARPASS_KEY);
    +    }
    +
    --- End diff --
    
    I would tend to agree there. Why not `PrivateKey`?


---