You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airavata.apache.org by am...@apache.org on 2013/03/21 21:09:44 UTC
svn commit: r1459504 -
/airavata/trunk/modules/security/src/main/java/org/apache/airavata/security/userstore/AbstractJDBCUserStore.java
Author: amilaj
Date: Thu Mar 21 20:09:43 2013
New Revision: 1459504
URL: http://svn.apache.org/r1459504
Log:
Fixing issue AIRAVATA-742
Modified:
airavata/trunk/modules/security/src/main/java/org/apache/airavata/security/userstore/AbstractJDBCUserStore.java
Modified: airavata/trunk/modules/security/src/main/java/org/apache/airavata/security/userstore/AbstractJDBCUserStore.java
URL: http://svn.apache.org/viewvc/airavata/trunk/modules/security/src/main/java/org/apache/airavata/security/userstore/AbstractJDBCUserStore.java?rev=1459504&r1=1459503&r2=1459504&view=diff
==============================================================================
--- airavata/trunk/modules/security/src/main/java/org/apache/airavata/security/userstore/AbstractJDBCUserStore.java (original)
+++ airavata/trunk/modules/security/src/main/java/org/apache/airavata/security/userstore/AbstractJDBCUserStore.java Thu Mar 21 20:09:43 2013
@@ -21,8 +21,13 @@
package org.apache.airavata.security.userstore;
+import org.apache.airavata.common.exception.ApplicationSettingsException;
+import org.apache.airavata.common.utils.ApplicationSettings;
+import org.apache.airavata.common.utils.ServerSettings;
import org.apache.airavata.security.UserStore;
import org.apache.airavata.security.UserStoreException;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import org.w3c.dom.Element;
import org.w3c.dom.Node;
import org.w3c.dom.NodeList;
@@ -32,6 +37,8 @@ import org.w3c.dom.NodeList;
*/
public abstract class AbstractJDBCUserStore implements UserStore {
+ protected static Logger log = LoggerFactory.getLogger(JDBCUserStore.class);
+
private String databaseURL = null;
private String databaseDriver = null;
private String databaseUserName = null;
@@ -55,9 +62,8 @@ public abstract class AbstractJDBCUserSt
/**
* Configures primary JDBC parameters. i.e
- *
- * @param node
- * An XML configuration node.
+ *
+ * @param node An XML configuration node.
* @throws UserStoreException
*/
public void configure(Node node) throws UserStoreException {
@@ -103,5 +109,26 @@ public abstract class AbstractJDBCUserSt
}
}
+ if (databaseURL == null || databaseUserName == null || databasePassword == null) {
+ // If database configurations are not specified in authenticators.xml we will read them from
+ // server.properties file.
+ try {
+ databaseDriver = ServerSettings.getCredentialStoreDBDriver();
+ databaseURL = ServerSettings.getCredentialStoreDBURL();
+ databaseUserName = ServerSettings.getCredentialStoreDBUser();
+ databasePassword = ServerSettings.getCredentialStoreDBPassword();
+
+ } catch (ApplicationSettingsException e) {
+ log.error("Error reading default user store DB configurations.");
+ throw new UserStoreException(e);
+ }
+
+ StringBuilder stringBuilder = new StringBuilder("User store configurations - dbDriver - ");
+ stringBuilder.append(databaseDriver);
+ stringBuilder.append(" URL - ").append(databaseURL).append(" DB user - ").append(databaseUserName);
+ log.info(stringBuilder.toString());
+
+ }
+
}
}