Skip to content

Commit a7ddacc

Browse files
committed
hopefully prevent a crash
1 parent 97722b7 commit a7ddacc

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

v3/opt_togetherjs/server.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,7 @@ var server = http.createServer(function(request, response) {
233233
request.headers['x-forwarded-for'] ||
234234
request.connection.remoteAddress ||
235235
request.socket.remoteAddress ||
236-
request.connection.socket.remoteAddress;
236+
(req.connection.socket ? req.connection.socket.remoteAddress ? null);
237237
uniqueId = 'IP_' + ip;
238238
}
239239
allRecentHelpQueueQueries.add(uniqueId);
@@ -496,7 +496,7 @@ wsServer.on('request', function(request) {
496496
request.headers['x-forwarded-for'] ||
497497
request.connection.remoteAddress ||
498498
request.socket.remoteAddress ||
499-
request.connection.socket.remoteAddress;
499+
(req.connection.socket ? req.connection.socket.remoteAddress ? null);
500500
uniqueId = 'IP_' + ip;
501501
}
502502
if (bannedUsers.indexOf(uniqueId) < 0) {
@@ -516,7 +516,8 @@ wsServer.on('request', function(request) {
516516
request.headers['x-forwarded-for'] ||
517517
request.connection.remoteAddress ||
518518
request.socket.remoteAddress ||
519-
request.connection.socket.remoteAddress;
519+
(req.connection.socket ? req.connection.socket.remoteAddress ? null);
520+
520521

521522
requestFunc("http://freegeoip.net/json/" + String(ip), function(error, resp, body) {
522523
var geoResult;
@@ -722,7 +723,7 @@ function createLogEntry(req, event_type) {
722723
req.headers['x-forwarded-for'] ||
723724
req.connection.remoteAddress ||
724725
req.socket.remoteAddress ||
725-
req.connection.socket.remoteAddress;
726+
(req.connection.socket ? req.connection.socket.remoteAddress ? null);
726727

727728
obj.ip = ip;
728729
obj.date = (new Date()).toISOString();

0 commit comments

Comments
 (0)