Fixes to the configuration object (#6)

* Fix env configuration type

* Required fields in config are no longer required

* Remove wonky changing of password attribute

* Update postgresql.html

Co-authored-by: Alexandre Alapetite <[email protected]>

Author: tedeh

postgresql.html

@@ -107,70 +107,60 @@
 			},
 			host: {
 				value: '127.0.0.1',
-				required: true,
 			},
 			hostFieldType: {
 				value: 'str',
 			},
 			port: {
 				value: 5432,
-				required: true,
 			},
 			portFieldType: {
 				value: 'num',
 			},
 			database: {
 				value: 'postgres',
-				required: true,
 			},
 			databaseFieldType: {
 				value: 'str',
 			},
 			ssl: {
 				value: false,
-				required: true,
 			},
 			sslFieldType: {
 				value: 'bool',
 			},
 			max: {
 				value: 10,
-				required: true,
 			},
 			maxFieldType: {
 				value: 'num',
 			},
 			min: {
 				value: 1,
-				required: true,
 			},
 			minFieldType: {
 				value: 'num',
 			},
 			idle: {
 				value: 1000,
-				required: true,
 			},
 			idleFieldType: {
 				value: 'num',
 			},
 			connectionTimeout: {
 				value: 10000,
-				required: true,
 			},
 			connectionTimeoutFieldType: {
 				value: 'num',
 			},
 			user: {
 				value: '',
-				required: true,
 			},
 			userFieldType: {
 				value: 'str',
 			},
 			password: {
 				value: '',
-				required: true,
 			},
 			passwordFieldType: {
 				value: 'str',
@@ -235,15 +225,6 @@
 					default: 'str',
 					types: ['str', 'flow', 'global', 'env'],
 					typeField: $('#node-config-input-passwordFieldType'),
-				})
-				.on('change', function (type, value) {
-					const e = $(this);
-					const kind = value === true ? e.typedInput('type') : value;
-					if (kind === 'str') {
-						e.attr('type', 'password');
-					} else {
-						e.attr('type', 'text');
-					}
 				});
 			$('#node-config-input-max').typedInput({
 				default: 'num',

postgresql.js

@@ -42,6 +42,8 @@ module.exports = function (RED) {
 				return parseInt(value);
 			case 'bool':
 				return JSON.parse(value);
+			case 'env':
+				return process.env[value];
 			default:
 				return value;
 		}