From 294d2acb6f1f1bf58ed270b9f2efbca548b8c29b Mon Sep 17 00:00:00 2001 From: Fabian Jakobs Date: Tue, 20 Dec 2016 13:55:19 +0000 Subject: [PATCH] generating uid is only needed on the server --- node_modules/kaefer/lib/server.js | 13 +++++++++++-- node_modules/kaefer/lib/util.js | 28 ---------------------------- 2 files changed, 11 insertions(+), 30 deletions(-) diff --git a/node_modules/kaefer/lib/server.js b/node_modules/kaefer/lib/server.js index bd3de993..9dcbe169 100644 --- a/node_modules/kaefer/lib/server.js +++ b/node_modules/kaefer/lib/server.js @@ -22,6 +22,7 @@ var util = require("./util"); var EventEmitter = require("events").EventEmitter; +var crypto = require("crypto"); var ReliableSocket = require("./reliable_socket"); var ReconnectSocket = require("./reconnect_socket"); @@ -78,7 +79,7 @@ Server.prototype.getConnection = function(id) { } } - id = util.uid(10); + id = generateUid(10); transport = this.sockets[id] = new ReliableSocket(new ReconnectSocket(), this.socketOptions); transport.on("disconnect", this.disconnect.bind(this, transport)); transport.id = id; @@ -92,4 +93,12 @@ Server.prototype.disconnect = function(transport) { transport.close(); this.emit("disconnect", transport); } -}; \ No newline at end of file +}; + +function generateUid(length) { + return (crypto + .randomBytes(length) + .toString("base64") + .slice(0, length) + ); +} diff --git a/node_modules/kaefer/lib/util.js b/node_modules/kaefer/lib/util.js index 772faba7..3c9c5784 100644 --- a/node_modules/kaefer/lib/util.js +++ b/node_modules/kaefer/lib/util.js @@ -4,32 +4,4 @@ exports.inherits = function(Child, Parent) { Child.prototype = Object.create(Parent.prototype, { constructor: { value: Child }}); }; -var isNode = (typeof global !== "undefined" && ({}).toString.call(global) == '[object global]'); -var isBrowser = (typeof window !== "undefined" && ({}).toString.call(window) == '[object window]'); - -if (isNode) { - exports.uid = function(length) { - // make sure packagers don't try to package "crypto" - return (require("cry" + "pto") - .randomBytes(length) - .toString("base64") - .slice(0, length) - ); - }; -} else if (isBrowser && window.crypto && window.crypto.getRandomValues) { - exports.uid = function(length) { - var values = new Uint8Array(length); - window.crypto.getRandomValues(values); - return btoa(String.fromCharCode.apply(null, values)).slice(0, length); - }; -} else { - exports.uid = function(length) { - var arr = []; - for (var i = 0; i < length; i++) { - arr[i] = Math.floor(Math.random() * 255); - } - return btoa(String.fromCharCode.apply(null, arr)).slice(0, length); - }; -} - });