فهرست منبع

fix delete endpoint

Tim Alby 8 سال پیش
والد
کامیت
60009c6346

+ 1 - 1
services/web/app/coffee/Features/User/UserEmailsController.coffee

@@ -33,7 +33,7 @@ module.exports = UserEmailsController =
 
 
 	remove: (req, res, next) ->
 	remove: (req, res, next) ->
 		userId = AuthenticationController.getLoggedInUserId(req)
 		userId = AuthenticationController.getLoggedInUserId(req)
-		email = EmailHelper.parseEmail(req.params.email)
+		email = EmailHelper.parseEmail(req.body.email)
 		return res.sendStatus 422 unless email?
 		return res.sendStatus 422 unless email?
 
 
 		UserUpdater.removeEmailAddress userId, email, (error)->
 		UserUpdater.removeEmailAddress userId, email, (error)->

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

@@ -114,7 +114,7 @@ module.exports = class Router
 		webRouter.post '/user/emails',
 		webRouter.post '/user/emails',
 			AuthenticationController.requireLogin(),
 			AuthenticationController.requireLogin(),
 			UserEmailsController.add
 			UserEmailsController.add
-		webRouter.delete '/user/emails/:email',
+		webRouter.post '/user/emails/delete',
 			AuthenticationController.requireLogin(),
 			AuthenticationController.requireLogin(),
 			UserEmailsController.remove
 			UserEmailsController.remove
 		webRouter.post '/user/emails/default',
 		webRouter.post '/user/emails/default',

+ 3 - 3
services/web/public/coffee/main/affiliations/factories/UserAffiliationsDataService.coffee

@@ -78,9 +78,9 @@ define [
 			}
 			}
 
 
 		removeUserEmail = (email) ->
 		removeUserEmail = (email) ->
-			$http.delete "/user/emails/#{ encodeURIComponent(email) }", {
-				headers: 
-					"X-CSRF-Token": window.csrfToken
+			$http.post "/user/emails/delete", {
+				email,
+				_csrf: window.csrfToken
 			}
 			}
 
 
 		isDomainBlacklisted = (domain) ->
 		isDomainBlacklisted = (domain) ->