From 6606375cd762ab0aeca9b0d45a3f0a1c4923d784 Mon Sep 17 00:00:00 2001
From: Shane Kilkelly <shane@kilkelly.me>
Date: Thu, 17 Nov 2016 15:27:58 +0000
Subject: [PATCH] Remove dead code: `SessionInvalidator`

---
 .../Security/SessionInvalidator.coffee        | 26 ---------
 .../Security/SessionInvalidatorTests.coffee   | 57 -------------------
 2 files changed, 83 deletions(-)
 delete mode 100644 services/web/app/coffee/Features/Security/SessionInvalidator.coffee
 delete mode 100644 services/web/test/UnitTests/coffee/Security/SessionInvalidatorTests.coffee

diff --git a/services/web/app/coffee/Features/Security/SessionInvalidator.coffee b/services/web/app/coffee/Features/Security/SessionInvalidator.coffee
deleted file mode 100644
index 668019f6f2..0000000000
--- a/services/web/app/coffee/Features/Security/SessionInvalidator.coffee
+++ /dev/null
@@ -1,26 +0,0 @@
-Settings = require("settings-sharelatex")
-redis = require("redis-sharelatex")
-rclient = redis.createClient(Settings.redis.web)
-crypto = require("crypto")
-async = require("async")
-
-
-module.exports = 
-
-	_getEmailKey : (email)->
-		hash = crypto.createHash("md5").update(email).digest("hex")
-		return "e_sess:#{hash}"
-
-	tracksession:(sessionId, email, callback = ->)->
-		session_lookup_key = @_getEmailKey(email)
-		rclient.set session_lookup_key, sessionId, callback
-
-	invalidateSession:(email, callback = ->)->
-		session_lookup_key = @_getEmailKey(email)
-		rclient.get session_lookup_key, (err, sessionId)->
-			async.series [
-				(cb)-> rclient.del sessionId, cb
-				(cb)-> rclient.del session_lookup_key, cb
-			], callback
-
-
diff --git a/services/web/test/UnitTests/coffee/Security/SessionInvalidatorTests.coffee b/services/web/test/UnitTests/coffee/Security/SessionInvalidatorTests.coffee
deleted file mode 100644
index 1a4550c248..0000000000
--- a/services/web/test/UnitTests/coffee/Security/SessionInvalidatorTests.coffee
+++ /dev/null
@@ -1,57 +0,0 @@
-should = require('chai').should()
-SandboxedModule = require('sandboxed-module')
-assert = require('assert')
-path = require('path')
-sinon = require('sinon')
-modulePath = path.join __dirname, "../../../../app/js/Features/Security/SessionInvalidator"
-expect = require("chai").expect
-
-describe "SessionInvaildator", ->
-
-	beforeEach ->
-		@settings =
-			redis:
-				web:{}
-		@rclient =
-			del:sinon.stub()
-			set:sinon.stub().callsArgWith(2)
-			get:sinon.stub()
-		@SessionInvaildator = SandboxedModule.require modulePath, requires:
-			"settings-sharelatex":@settings
-			"logger-sharelatex": log:->
-			"redis-sharelatex": createClient:=> 
-				return @rclient
-		@emailAddress = "bob@smith"
-		@sessionId = "sess:123456"
-		@stubbedKey = "e_sess:7890"
-
-	describe "_getEmailKey", ->
-
-		it "should get the email key by hashing it", ->
-			result = @SessionInvaildator._getEmailKey "bob@smith.com"
-			result.should.equal "e_sess:6815b961bfb8f83dd4cecd357e55e62d"
-
-	describe "tracksession", ->
-
-		it "should save the session in redis", (done)->
-
-			@SessionInvaildator._getEmailKey = sinon.stub().returns(@stubbedKey)
-			@SessionInvaildator.tracksession @sessionId, @emailAddress, =>
-				@rclient.set.calledWith(@stubbedKey).should.equal true
-				done()
-
-
-	describe "invalidateSession", (done)->
-
-		beforeEach ->
-			@SessionInvaildator._getEmailKey = sinon.stub().returns(@stubbedKey)
-			@rclient.del.callsArgWith(1)
-
-		it "get the session key and delete it", (done)->
-			@rclient.get.callsArgWith 1, null, @sessionId
-			@SessionInvaildator.invalidateSession @emailAddress, =>
-				@rclient.del.calledWith(@sessionId).should.equal true
-				@rclient.del.calledWith(@stubbedKey).should.equal true
-
-				done()
-