java/sql-dk/src/info/globalcode/sql/dk/DatabaseConnection.java
branchv_0
changeset 108 d06d90b28217
parent 107 8189a4a28cd8
child 142 da1e38386d84
--- a/java/sql-dk/src/info/globalcode/sql/dk/DatabaseConnection.java	Wed Jan 01 02:44:29 2014 +0100
+++ b/java/sql-dk/src/info/globalcode/sql/dk/DatabaseConnection.java	Wed Jan 01 02:56:08 2014 +0100
@@ -39,6 +39,8 @@
 public class DatabaseConnection implements AutoCloseable {
 
 	private static final Logger log = Logger.getLogger(DatabaseConnection.class.getName());
+	private static final String JDBC_PROPERTY_USER = "user";
+	public static final String JDBC_PROPERTY_PASSWORD = "password";
 	private DatabaseDefinition databaseDefinition;
 	private Connection connection;
 	private Properties properties;
@@ -47,9 +49,13 @@
 		this.databaseDefinition = databaseDefinition;
 		this.properties = properties;
 
+		if (properties.hasProperty(JDBC_PROPERTY_PASSWORD)) {
+			log.log(Level.WARNING, "Passing DB password as CLI parameter is insecure!");
+		}
+
 		Properties credentials = new Properties();
-		credentials.add(new Property("user", databaseDefinition.getUserName()));
-		credentials.add(new Property("password", databaseDefinition.getPassword()));
+		credentials.add(new Property(JDBC_PROPERTY_USER, databaseDefinition.getUserName()));
+		credentials.add(new Property(JDBC_PROPERTY_PASSWORD, databaseDefinition.getPassword()));
 		credentials.setDefaults(databaseDefinition.getProperties());
 		properties.setDefaults(credentials);
 		java.util.Properties javaProperties = properties.getJavaProperties();