Skip to content

Commit 38d639a

Browse files
markbjerkedarrachequesne
authored andcommittedSep 2, 2017
[fix] Use explicit require of wsEngine (#523)
Because WebPack does not support variables in require statements. Closes #522
1 parent f9d3f06 commit 38d639a

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed
 

‎lib/server.js

+5-1
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,11 @@ Server.prototype.init = function () {
104104

105105
var wsModule;
106106
try {
107-
wsModule = require(this.wsEngine);
107+
switch (this.wsEngine) {
108+
case 'uws': wsModule = require('uws'); break;
109+
case 'ws': wsModule = require('ws'); break;
110+
default: throw new Error('unknown wsEngine');
111+
}
108112
} catch (ex) {
109113
this.wsEngine = 'ws';
110114
// keep require('ws') as separate expression for packers (browserify, etc)

0 commit comments

Comments
 (0)
Please sign in to comment.