Github user necouchman commented on a diff in the pull request:
https://github.com/apache/incubator-guacamole-client/pull/183#discussion_r135426332
--- 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 --
Taking a key argument and calling this a cipher property feels a little
kludgy. Kind of goes along with the question about how far the Cipher property
should go or if that's even really a good name/function for it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
