From 38d639a5d0b0b6d679a17dbafbed386aad6232df Mon Sep 17 00:00:00 2001 From: Mark Bjerke Date: Sat, 2 Sep 2017 05:12:26 -0700 Subject: [PATCH] [fix] Use explicit require of wsEngine (#523) Because WebPack does not support variables in require statements. Closes #522 --- lib/server.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/server.js b/lib/server.js index 57886b596..f41d59eab 100644 --- a/lib/server.js +++ b/lib/server.js @@ -104,7 +104,11 @@ Server.prototype.init = function () { var wsModule; try { - wsModule = require(this.wsEngine); + switch (this.wsEngine) { + case 'uws': wsModule = require('uws'); break; + case 'ws': wsModule = require('ws'); break; + default: throw new Error('unknown wsEngine'); + } } catch (ex) { this.wsEngine = 'ws'; // keep require('ws') as separate expression for packers (browserify, etc)