Sfoglia il codice sorgente

Remove deprecated endpoint for old delete-user

Shane Kilkelly 10 anni fa
parent
commit
dc62b1a86e

+ 0 - 8
services/web/app/coffee/Features/User/UserController.coffee

@@ -15,14 +15,6 @@ settings = require "settings-sharelatex"
 
 
 module.exports = UserController =
 module.exports = UserController =
 
 
-	# TODO: deprecated, remove
-	deleteUser: (req, res)->
-		user_id = AuthenticationController.getLoggedInUserId(req)
-		UserDeleter.deleteUser user_id, (err)->
-			if !err?
-				req.session?.destroy()
-			res.sendStatus(200)
-
 	tryDeleteUser: (req, res, next) ->
 	tryDeleteUser: (req, res, next) ->
 		user_id = AuthenticationController.getLoggedInUserId(req)
 		user_id = AuthenticationController.getLoggedInUserId(req)
 		password = req.body.password
 		password = req.body.password

+ 0 - 1
services/web/app/coffee/router.coffee

@@ -92,7 +92,6 @@ module.exports = class Router
 		webRouter.post '/user/sessions/clear', AuthenticationController.requireLogin(), UserController.clearSessions
 		webRouter.post '/user/sessions/clear', AuthenticationController.requireLogin(), UserController.clearSessions
 
 
 		webRouter.delete '/user/newsletter/unsubscribe', AuthenticationController.requireLogin(), UserController.unsubscribe
 		webRouter.delete '/user/newsletter/unsubscribe', AuthenticationController.requireLogin(), UserController.unsubscribe
-		webRouter.delete '/user', AuthenticationController.requireLogin(), UserController.deleteUser
 		webRouter.post '/user/delete', AuthenticationController.requireLogin(), UserController.tryDeleteUser
 		webRouter.post '/user/delete', AuthenticationController.requireLogin(), UserController.tryDeleteUser
 
 
 		webRouter.get  '/user/personal_info', AuthenticationController.requireLogin(), UserInfoController.getLoggedInUsersPersonalInfo
 		webRouter.get  '/user/personal_info', AuthenticationController.requireLogin(), UserInfoController.getLoggedInUsersPersonalInfo

+ 0 - 10
services/web/test/UnitTests/coffee/User/UserControllerTests.coffee

@@ -85,16 +85,6 @@ describe "UserController", ->
 			json: sinon.stub()
 			json: sinon.stub()
 		@next = sinon.stub()
 		@next = sinon.stub()
 
 
-	describe "deleteUser", ->
-
-		it "should delete the user", (done)->
-
-			@res.sendStatus = (code)=>
-				@UserDeleter.deleteUser.calledWith(@user_id)
-				code.should.equal 200
-				done()
-			@UserController.deleteUser @req, @res
-
 	describe 'tryDeleteUser', ->
 	describe 'tryDeleteUser', ->
 
 
 		beforeEach ->
 		beforeEach ->