Merge pull request #187 from Amolith/master
remove legacy code to solve #58
This commit is contained in:
@@ -3,7 +3,6 @@
|
|||||||
// external modules
|
// external modules
|
||||||
var cookie = require('cookie')
|
var cookie = require('cookie')
|
||||||
var cookieParser = require('cookie-parser')
|
var cookieParser = require('cookie-parser')
|
||||||
var url = require('url')
|
|
||||||
var async = require('async')
|
var async = require('async')
|
||||||
var randomcolor = require('randomcolor')
|
var randomcolor = require('randomcolor')
|
||||||
var Chance = require('chance')
|
var Chance = require('chance')
|
||||||
@@ -277,16 +276,6 @@ function extractNoteIdFromSocket (socket) {
|
|||||||
}
|
}
|
||||||
if (socket.handshake.query && socket.handshake.query.noteId) {
|
if (socket.handshake.query && socket.handshake.query.noteId) {
|
||||||
return socket.handshake.query.noteId
|
return socket.handshake.query.noteId
|
||||||
} else if (socket.handshake.headers) {
|
|
||||||
// this part is only for backward compatibility only; current code
|
|
||||||
// should be using noteId query parameter instead.
|
|
||||||
var referer = socket.handshake.headers.referer
|
|
||||||
if (!referer) {
|
|
||||||
return false
|
|
||||||
}
|
|
||||||
var hostUrl = url.URL.parse(referer)
|
|
||||||
var noteId = config.urlPath ? hostUrl.pathname.slice(config.urlPath.length + 1, hostUrl.pathname.length).split('/')[1] : hostUrl.pathname.split('/')[1]
|
|
||||||
return noteId
|
|
||||||
} else {
|
} else {
|
||||||
return false
|
return false
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user