Pass abort reason of thunk action to AbortController

[xbaun]

Hi,

I noticed that the abort reason of a thunk action is not getting passed down to the AbortController in line 578:

redux-toolkit/packages/toolkit/src/createAsyncThunk.ts

Lines 575 to 580 in 64a30d8

	function abort(reason?: string) {
	if (started) {
	abortReason = reason
	abortController.abort()
	}
	}

It would be useful to get the abort reason directly in the thunk action, so the thunk can cleanup its states.

thx

[phryneas]

AbortController only added support for a reason recently and it is spotty across different platforms. If you can come up with a good overview on where it is supported and where it might cause problems passing it in we could get to an informed decision, but I don't have the time to do that research atm.

[xbaun]

There is a MDN overview for reason where it is supported https://developer.mozilla.org/en-US/docs/Web/API/AbortSignal/reason#browser_compatibility - looks pretty good.

It's also supported on node with version v16 and above: https://nodejs.org/docs/latest-v16.x/api/globals.html#abortcontrollerabortreason

Are there other platforms do you mean?

[FaberVitale]

AbortController only added support for a reason recently and it is spotty across different platforms. If you can come up with a good overview on where it is supported and where it might cause problems passing it in we could get to an informed decision, but I don't have the time to do that research atm.

In listenerMiddleware code, reason is patched if it is not avaible at runtime:

redux-toolkit/packages/toolkit/src/listenerMiddleware/utils.ts

Line 43 in 2b80d8b

export const abortControllerWithReason = <T>(

We could reuse abortControllerWithReason in asyncThunks