Skip to content

Commit 8592be7

Browse files
committed
Merge pull request #604 from markyen/socket-port
Include port as a connection parameter for socket connections
2 parents d9b72f5 + bf26569 commit 8592be7

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

.travis.yml

+2
Original file line numberDiff line numberDiff line change
@@ -3,3 +3,5 @@ node_js:
33
- "0.10"
44
before_script:
55
- node script/create-test-tables.js pg://[email protected]:5432/postgres
6+
before_install:
7+
- npm install -g npm@~1.4.6

lib/connection-parameters.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ var parse = function(str) {
3939
config.fallback_application_name = result.query.fallback_application_name;
4040
}
4141

42+
config.port = result.port;
4243
if(result.protocol == 'socket:') {
4344
config.host = decodeURI(result.pathname);
4445
config.database = result.query.db;
@@ -50,7 +51,6 @@ var parse = function(str) {
5051
var auth = (result.auth || ':').split(':');
5152
config.user = auth[0];
5253
config.password = auth[1];
53-
config.port = result.port;
5454

5555
var ssl = result.query.ssl;
5656
if (ssl === 'true' || ssl === '1') {
@@ -66,7 +66,7 @@ var useSsl = function() {
6666
return false;
6767
case "prefer":
6868
case "require":
69-
case "verify-ca":
69+
case "verify-ca":
7070
case "verify-full":
7171
return true;
7272
}

0 commit comments

Comments
 (0)