fix: csrf token only on non-GET routes

NodeBB · Oct 14, 2020 · 20bb9c7 · 20bb9c7
1 parent 3326d80
commit 20bb9c7
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/public/src/modules/api.js b/public/src/modules/api.js
@@ -10,11 +10,7 @@ define('api', () => {
 			baseUrl + options.url;
 
 		function doAjax(cb) {
-			$.ajax(Object.assign({
-				headers: {
-					'x-csrf-token': config.csrf_token,
-				},
-			}, options))
+			$.ajax(options)
 				.done((res) => {
 					cb(null,
 						res.hasOwnProperty('status') && res.hasOwnProperty('response') ?
@@ -57,18 +53,27 @@ define('api', () => {
 		url: route,
 		method: 'post',
 		data: payload,
+		headers: {
+			'x-csrf-token': config.csrf_token,
+		},
 	}, onSuccess);
 
 	api.put = (route, payload, onSuccess) => call({
 		url: route,
 		method: 'put',
 		data: payload,
+		headers: {
+			'x-csrf-token': config.csrf_token,
+		},
 	}, onSuccess);
 
 	api.del = (route, payload, onSuccess) => call({
 		url: route,
 		method: 'delete',
 		data: payload,
+		headers: {
+			'x-csrf-token': config.csrf_token,
+		},
 	}, onSuccess);
 
 	return api;