nicdex/node-eventstore-client
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

14 - fix creationConnection with tcp://host:port failing

Browse Source
This commit is contained in:
Nicolas Dextraze 2016-12-06 18:54:04 -08:00
parent 52dee3584e
commit 3acfaafb2a
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/eventStoreConnection.js
View File

@ -56,7 +56,7 @@ function createFromTcpEndpoint(settings, tcpEndpoint, connectionName) {
} }
function createFromStringEndpoint(settings, endPoint, connectionName) { function createFromStringEndpoint(settings, endPoint, connectionName) {
var m = endPoint.match(/^(tcp|discover):\/\/([^:]):?(\d+)?$/); var m = endPoint.match(/^(tcp|discover):\/\/([^:]+):?(\d+)?$/);
if (!m) throw new Error('endPoint string must be tcp://hostname[:port] or discover://dns[:port]'); if (!m) throw new Error('endPoint string must be tcp://hostname[:port] or discover://dns[:port]');
var scheme = m[1]; var scheme = m[1];
var hostname = m[2]; var hostname = m[2];

5
test/connection_test.js
View File

@ -41,6 +41,11 @@ module.exports = {
test.ok(reason.indexOf("Reconnection limit reached") === 0, "Wrong expected reason."); test.ok(reason.indexOf("Reconnection limit reached") === 0, "Wrong expected reason.");
test.done(); test.done();
}); });
},
'Create a connection with tcp://host:port string': function(test) {
var conn = client.createConnection({}, 'tcp://localhost:2113');
conn.close();
test.done();
}/*, }/*,
'Connect to Cluster using gossip seeds': function (test) { 'Connect to Cluster using gossip seeds': function (test) {
test.expect(1); test.expect(1);