Merge pull request #1250 from nextcloud/backport/1247/stable23

[stable23] Use modern API to cancel requests
nextcloud · May 20, 2022 · a78c2b4 · a78c2b4
2 parents 3fa859e + 2478dc7
commit a78c2b4
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 12 deletions.
diff --git a/js/viewer-main.js b/js/viewer-main.js
diff --git a/js/viewer-main.js.LICENSE.txt b/js/viewer-main.js.LICENSE.txt
@@ -183,6 +183,7 @@ object-assign
  *
  * @author Marco Ambrosini <marcoambrosini@pm.me>
  * @author John Molakvoæ <skjnldsv@protonmail.com>
+ * @author Louis Chemineau <louis@chmn.me>
  *
  * @license GNU AGPL version 3 or any later version
  *

diff --git a/js/viewer-main.js.map b/js/viewer-main.js.map
diff --git a/src/utils/CancelableRequest.js b/src/utils/CancelableRequest.js
@@ -3,6 +3,7 @@
  *
  * @author Marco Ambrosini <marcoambrosini@pm.me>
  * @author John Molakvoæ <skjnldsv@protonmail.com>
+ * @author Louis Chemineau <louis@chmn.me>
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -21,20 +22,14 @@
  *
  */
 
-import axios from '@nextcloud/axios'
-
 /**
  * Creates a cancelable axios 'request object'.
  *
  * @param {function} request the axios promise request
  * @returns {Object}
  */
 const CancelableRequest = function(request) {
-	/**
-	 * Generate an axios cancel token
-	 */
-	const CancelToken = axios.CancelToken
-	const source = CancelToken.source()
+	const controller = new AbortController()
 
 	/**
 	 * Execute the request
@@ -45,12 +40,12 @@ const CancelableRequest = function(request) {
 	const fetch = async function(url, options) {
 		return request(
 			url,
-			Object.assign({ cancelToken: source.token }, { options })
+			{ ...options, signal: controller.signal }
 		)
 	}
 	return {
 		request: fetch,
-		cancel: source.cancel,
+		cancel: () => controller.abort(),
 	}
 }