From 6063d7e41c021512c985c104bfc0b7e5c5e9288e Mon Sep 17 00:00:00 2001
From: Jason Dusek <jason.dusek@gmail.com>
Date: Fri, 25 May 2018 19:51:58 -0700
Subject: [PATCH] Tie default DB name to database user name

Before, the default database name was tied to the system user name, which differs from the behavior of the native client.
---
 lib/connection-parameters.js | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/lib/connection-parameters.js b/lib/connection-parameters.js
index f31f28dd3..d61accad6 100644
--- a/lib/connection-parameters.js
+++ b/lib/connection-parameters.js
@@ -13,7 +13,7 @@ var defaults = require('./defaults')
 
 var parse = require('pg-connection-string').parse // parses a connection string
 
-var val = function (key, config, envVar) {
+var val = function (key, config, envVar, computed) {
   if (envVar === undefined) {
     envVar = process.env[ 'PG' + key.toUpperCase() ]
   } else if (envVar === false) {
@@ -22,9 +22,13 @@ var val = function (key, config, envVar) {
     envVar = process.env[ envVar ]
   }
 
+  if (computed === undefined) {
+    computed = function (key) { return defaults[key] }
+  }
+
   return config[key] ||
     envVar ||
-    defaults[key]
+    computed(key)
 }
 
 var useSsl = function () {
@@ -50,8 +54,11 @@ var ConnectionParameters = function (config) {
     config = Object.assign({}, config, parse(config.connectionString))
   }
 
+  var dbDefaulting = function () { return this.user || defaults['database'] }
+  dbDefaulting = dbDefaulting.bind(this)
+
   this.user = val('user', config)
-  this.database = val('database', config)
+  this.database = val('database', config, undefined, dbDefaulting)
   this.port = parseInt(val('port', config), 10)
   this.host = val('host', config)
   this.password = val('password', config)