You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@guacamole.apache.org by mj...@apache.org on 2018/12/05 22:40:09 UTC
[1/3] guacamole-client git commit: GUACAMOLE-526: Fix loading of CAS
ticket field and redirection.
Repository: guacamole-client
Updated Branches:
refs/heads/staging/1.0.0 a967ffa41 -> 801a5df9f
GUACAMOLE-526: Fix loading of CAS ticket field and redirection.
Project: http://git-wip-us.apache.org/repos/asf/guacamole-client/repo
Commit: http://git-wip-us.apache.org/repos/asf/guacamole-client/commit/df8c07c1
Tree: http://git-wip-us.apache.org/repos/asf/guacamole-client/tree/df8c07c1
Diff: http://git-wip-us.apache.org/repos/asf/guacamole-client/diff/df8c07c1
Branch: refs/heads/staging/1.0.0
Commit: df8c07c18755d0742df20d2e66c5e6d5881129d8
Parents: a967ffa
Author: Nick Couchman <vn...@apache.org>
Authored: Sun Dec 2 16:34:16 2018 -0500
Committer: Nick Couchman <vn...@apache.org>
Committed: Tue Dec 4 19:28:14 2018 -0500
----------------------------------------------------------------------
.../auth/cas/AuthenticationProviderService.java | 12 ++----------
.../apache/guacamole/auth/cas/form/CASTicketField.java | 2 --
2 files changed, 2 insertions(+), 12 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/guacamole-client/blob/df8c07c1/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/AuthenticationProviderService.java
----------------------------------------------------------------------
diff --git a/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/AuthenticationProviderService.java b/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/AuthenticationProviderService.java
index a9b3230..d769d2e 100644
--- a/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/AuthenticationProviderService.java
+++ b/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/AuthenticationProviderService.java
@@ -23,12 +23,11 @@ import com.google.inject.Inject;
import com.google.inject.Provider;
import java.util.Arrays;
import javax.servlet.http.HttpServletRequest;
-import org.apache.guacamole.environment.Environment;
import org.apache.guacamole.form.Field;
import org.apache.guacamole.GuacamoleException;
import org.apache.guacamole.net.auth.Credentials;
import org.apache.guacamole.net.auth.credentials.CredentialsInfo;
-import org.apache.guacamole.net.auth.credentials.GuacamoleInsufficientCredentialsException;
+import org.apache.guacamole.net.auth.credentials.GuacamoleInvalidCredentialsException;
import org.apache.guacamole.auth.cas.conf.ConfigurationService;
import org.apache.guacamole.auth.cas.form.CASTicketField;
import org.apache.guacamole.auth.cas.ticket.TicketValidationService;
@@ -47,12 +46,6 @@ public class AuthenticationProviderService {
private ConfigurationService confService;
/**
- * The Guacamole server environment.
- */
- @Inject
- private Environment environment;
-
- /**
* Service for validating received ID tickets.
*/
@Inject
@@ -97,8 +90,7 @@ public class AuthenticationProviderService {
}
// Request CAS ticket
- throw new GuacamoleInsufficientCredentialsException(
- "LOGIN.INFO_CAS_REDIRECT_PENDING",
+ throw new GuacamoleInvalidCredentialsException("Invalid login.",
new CredentialsInfo(Arrays.asList(new Field[] {
// CAS-specific ticket (will automatically redirect the user
http://git-wip-us.apache.org/repos/asf/guacamole-client/blob/df8c07c1/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/form/CASTicketField.java
----------------------------------------------------------------------
diff --git a/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/form/CASTicketField.java b/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/form/CASTicketField.java
index eb0f68e..f785241 100644
--- a/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/form/CASTicketField.java
+++ b/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/form/CASTicketField.java
@@ -20,9 +20,7 @@
package org.apache.guacamole.auth.cas.form;
import java.io.UnsupportedEncodingException;
-import java.math.BigInteger;
import java.net.URLEncoder;
-import java.security.SecureRandom;
import org.apache.guacamole.form.Field;
[2/3] guacamole-client git commit: GUACAMOLE-526: Clean up unused
code and minor errors.
Posted by mj...@apache.org.
GUACAMOLE-526: Clean up unused code and minor errors.
Project: http://git-wip-us.apache.org/repos/asf/guacamole-client/repo
Commit: http://git-wip-us.apache.org/repos/asf/guacamole-client/commit/29982e3a
Tree: http://git-wip-us.apache.org/repos/asf/guacamole-client/tree/29982e3a
Diff: http://git-wip-us.apache.org/repos/asf/guacamole-client/diff/29982e3a
Branch: refs/heads/staging/1.0.0
Commit: 29982e3a855fcec2fae4c0d37e2e0ad71b1ca726
Parents: df8c07c
Author: Nick Couchman <vn...@apache.org>
Authored: Tue Dec 4 19:16:57 2018 -0500
Committer: Nick Couchman <vn...@apache.org>
Committed: Tue Dec 4 19:28:18 2018 -0500
----------------------------------------------------------------------
.../apache/guacamole/auth/cas/conf/ConfigurationService.java | 1 -
.../guacamole/auth/cas/conf/PrivateKeyGuacamoleProperty.java | 5 -----
extensions/guacamole-auth-cas/src/main/resources/casModule.js | 3 +--
3 files changed, 1 insertion(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/guacamole-client/blob/29982e3a/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/ConfigurationService.java
----------------------------------------------------------------------
diff --git a/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/ConfigurationService.java b/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/ConfigurationService.java
index dee9c4d..e0016ad 100644
--- a/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/ConfigurationService.java
+++ b/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/ConfigurationService.java
@@ -20,7 +20,6 @@
package org.apache.guacamole.auth.cas.conf;
import com.google.inject.Inject;
-import java.io.File;
import java.security.PrivateKey;
import org.apache.guacamole.GuacamoleException;
import org.apache.guacamole.environment.Environment;
http://git-wip-us.apache.org/repos/asf/guacamole-client/blob/29982e3a/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/PrivateKeyGuacamoleProperty.java
----------------------------------------------------------------------
diff --git a/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/PrivateKeyGuacamoleProperty.java b/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/PrivateKeyGuacamoleProperty.java
index 455f177..d1c1685 100644
--- a/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/PrivateKeyGuacamoleProperty.java
+++ b/extensions/guacamole-auth-cas/src/main/java/org/apache/guacamole/auth/cas/conf/PrivateKeyGuacamoleProperty.java
@@ -23,10 +23,7 @@ import java.io.ByteArrayOutputStream;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
-import java.io.InputStream;
import java.io.IOException;
-import java.lang.IllegalArgumentException;
-import java.security.InvalidKeyException;
import java.security.KeyFactory;
import java.security.NoSuchAlgorithmException;
import java.security.PrivateKey;
@@ -35,8 +32,6 @@ import java.security.spec.KeySpec;
import java.security.spec.PKCS8EncodedKeySpec;
import org.apache.guacamole.properties.GuacamoleProperty;
import org.apache.guacamole.GuacamoleServerException;
-import org.apache.guacamole.environment.Environment;
-import org.apache.guacamole.environment.LocalEnvironment;
/**
* A GuacamoleProperty whose value is derived from a private key file.
http://git-wip-us.apache.org/repos/asf/guacamole-client/blob/29982e3a/extensions/guacamole-auth-cas/src/main/resources/casModule.js
----------------------------------------------------------------------
diff --git a/extensions/guacamole-auth-cas/src/main/resources/casModule.js b/extensions/guacamole-auth-cas/src/main/resources/casModule.js
index bd789dd..c40e6b2 100644
--- a/extensions/guacamole-auth-cas/src/main/resources/casModule.js
+++ b/extensions/guacamole-auth-cas/src/main/resources/casModule.js
@@ -21,8 +21,7 @@
* Module which provides handling for CAS authentication.
*/
angular.module('guacCAS', [
- 'form',
- 'ngRoute',
+ 'form'
]);
// Ensure the CAS module is loaded along with the rest of the app
[3/3] guacamole-client git commit: GUACAMOLE-526: Merge changes
correcting regression in CAS due to AngularJS update.
Posted by mj...@apache.org.
GUACAMOLE-526: Merge changes correcting regression in CAS due to AngularJS update.
Including `ngRoute` as a dependency of a module which defines a custom
field type will always result in the following error:
Unknown provider: $rootElementProvider <- $rootElement <- $location <- $route <- $location
This seems to be due to the manner in which the injector is created for
field types. `$rootElementProvider` is not available for injection in
the context of the injector of a custom field. This is generally OK, but
`ngRoute` transitively depends on `$rootElementProvider`. Since
`$rootElementProvider` cannot be injected within the context of a field
type, no field type can depend on ngRoute without causing this error.
Note that extensions can still leverage `ngRoute` (this is already done
by the OpenID support). It just cannot be declared as a dependency of a
module containing a custom field type.
Project: http://git-wip-us.apache.org/repos/asf/guacamole-client/repo
Commit: http://git-wip-us.apache.org/repos/asf/guacamole-client/commit/801a5df9
Tree: http://git-wip-us.apache.org/repos/asf/guacamole-client/tree/801a5df9
Diff: http://git-wip-us.apache.org/repos/asf/guacamole-client/diff/801a5df9
Branch: refs/heads/staging/1.0.0
Commit: 801a5df9f1d7095c52e594dda1a5276fe8cf6524
Parents: a967ffa 29982e3
Author: Michael Jumper <mj...@apache.org>
Authored: Wed Dec 5 14:32:37 2018 -0800
Committer: Michael Jumper <mj...@apache.org>
Committed: Wed Dec 5 14:32:37 2018 -0800
----------------------------------------------------------------------
.../auth/cas/AuthenticationProviderService.java | 12 ++----------
.../guacamole/auth/cas/conf/ConfigurationService.java | 1 -
.../auth/cas/conf/PrivateKeyGuacamoleProperty.java | 5 -----
.../apache/guacamole/auth/cas/form/CASTicketField.java | 2 --
.../guacamole-auth-cas/src/main/resources/casModule.js | 3 +--
5 files changed, 3 insertions(+), 20 deletions(-)
----------------------------------------------------------------------