GUACAMOLE-641: Migrate existing extensions to singleton LocalEnvironment.

This commit is contained in:
Michael Jumper
2020-08-25 16:25:42 -07:00
parent 3a24b2f556
commit 4d65105695
14 changed files with 18 additions and 32 deletions

View File

@@ -58,7 +58,7 @@ public class CASAuthenticationProviderModule extends AbstractModule {
throws GuacamoleException {
// Get local environment
this.environment = new LocalEnvironment();
this.environment = LocalEnvironment.getInstance();
// Store associated auth provider
this.authProvider = authProvider;

View File

@@ -58,7 +58,7 @@ public class DuoAuthenticationProviderModule extends AbstractModule {
throws GuacamoleException {
// Get local environment
this.environment = new LocalEnvironment();
this.environment = LocalEnvironment.getInstance();
// Store associated auth provider
this.authProvider = authProvider;

View File

@@ -56,7 +56,7 @@ public class HTTPHeaderAuthenticationProviderModule extends AbstractModule {
throws GuacamoleException {
// Get local environment
this.environment = new LocalEnvironment();
this.environment = LocalEnvironment.getInstance();
// Store associated auth provider
this.authProvider = authProvider;

View File

@@ -20,25 +20,23 @@
package org.apache.guacamole.auth.jdbc;
import org.apache.guacamole.GuacamoleException;
import org.apache.guacamole.environment.LocalEnvironment;
import org.apache.guacamole.auth.jdbc.security.PasswordPolicy;
import org.apache.guacamole.environment.DelegatingEnvironment;
import org.apache.guacamole.environment.LocalEnvironment;
import org.apache.ibatis.session.SqlSession;
/**
* A JDBC-specific implementation of Environment that defines generic properties
* intended for use within JDBC based authentication providers.
*/
public abstract class JDBCEnvironment extends LocalEnvironment {
public abstract class JDBCEnvironment extends DelegatingEnvironment {
/**
* Constructs a new JDBCEnvironment using an underlying LocalEnviroment to
* read properties from the file system.
*
* @throws GuacamoleException
* If an error occurs while setting up the underlying LocalEnvironment.
*/
public JDBCEnvironment() throws GuacamoleException {
super();
public JDBCEnvironment() {
super(LocalEnvironment.getInstance());
}
/**

View File

@@ -59,7 +59,7 @@ public class JSONAuthenticationProviderModule extends AbstractModule {
throws GuacamoleException {
// Get local environment
this.environment = new LocalEnvironment();
this.environment = LocalEnvironment.getInstance();
// Store associated auth provider
this.authProvider = authProvider;

View File

@@ -60,7 +60,7 @@ public class LDAPAuthenticationProviderModule extends AbstractModule {
throws GuacamoleException {
// Get local environment
this.environment = new LocalEnvironment();
this.environment = LocalEnvironment.getInstance();
// Store associated auth provider
this.authProvider = authProvider;

View File

@@ -59,7 +59,7 @@ public class OpenIDAuthenticationProviderModule extends AbstractModule {
throws GuacamoleException {
// Get local environment
this.environment = new LocalEnvironment();
this.environment = LocalEnvironment.getInstance();
// Store associated auth provider
this.authProvider = authProvider;

View File

@@ -57,7 +57,7 @@ public class QuickConnectAuthenticationProviderModule extends AbstractModule {
AuthenticationProvider authProvider) throws GuacamoleException {
// Get local environment
this.environment = new LocalEnvironment();
this.environment = LocalEnvironment.getInstance();
// Store associated auth provider
this.authProvider = authProvider;

View File

@@ -63,7 +63,7 @@ public class RadiusAuthenticationProviderModule extends AbstractModule {
throws GuacamoleException {
// Get local environment
this.environment = new LocalEnvironment();
this.environment = LocalEnvironment.getInstance();
// Check for MD4 requirement
RadiusAuthenticationProtocol authProtocol = environment.getProperty(RadiusGuacamoleProperties.RADIUS_AUTH_PROTOCOL);

View File

@@ -57,7 +57,7 @@ public class SAMLAuthenticationProviderModule extends AbstractModule {
throws GuacamoleException {
// Get local environment
this.environment = new LocalEnvironment();
this.environment = LocalEnvironment.getInstance();
// Store associated auth provider
this.authProvider = authProvider;

View File

@@ -59,7 +59,7 @@ public class TOTPAuthenticationProviderModule extends AbstractModule {
throws GuacamoleException {
// Get local environment
this.environment = new LocalEnvironment();
this.environment = LocalEnvironment.getInstance();
// Store associated auth provider
this.authProvider = authProvider;

View File

@@ -202,7 +202,7 @@ public class SimpleConnection extends AbstractConnection {
throws GuacamoleException {
// Retrieve proxy configuration from environment
Environment environment = new LocalEnvironment();
Environment environment = LocalEnvironment.getInstance();
GuacamoleProxyConfiguration proxyConfig = environment.getDefaultGuacamoleProxyConfiguration();
// Get guacd connection parameters

View File

@@ -63,25 +63,13 @@ public class FileAuthenticationProvider extends SimpleAuthenticationProvider {
/**
* Guacamole server environment.
*/
private final Environment environment;
private final Environment environment = LocalEnvironment.getInstance();
/**
* The filename to use for the user mapping.
*/
public static final String USER_MAPPING_FILENAME = "user-mapping.xml";
/**
* Creates a new FileAuthenticationProvider that authenticates users against
* simple, monolithic XML file.
*
* @throws GuacamoleException
* If a required property is missing, or an error occurs while parsing
* a property.
*/
public FileAuthenticationProvider() throws GuacamoleException {
environment = new LocalEnvironment();
}
@Override
public String getIdentifier() {
return "default";

View File

@@ -171,7 +171,7 @@ public class SchemaResource {
public Map<String, ProtocolInfo> getProtocols() throws GuacamoleException {
// Get and return a map of all protocols.
Environment env = new LocalEnvironment();
Environment env = LocalEnvironment.getInstance();
return env.getProtocols();
}