fromFetch cannot abort all responses

Bug Report Feature Request

Current Behavior

fromFetch returns an observable that emits the request’s Response object. When json() or text() is called on the Response, a promise is returned. My understanding is that the AbortController used in the fetch call also aborts promises returned by the aforementioned methods.

In #4742, a bug was fixed: fromFetch was calling abort as part of the implicit unsubscribe that occurs after completion. This would abort any still pending responses. When fetch resolves, a flag is now set to prevent abort being called in the unsubscribe teardown.

However, that means there is no way of aborting the promises returned by calls to Response methods.

Possible Solution

Add another signature to fromFetch so that a selector can be passed:

export function fromFetch<R>(
  input: string | Request,
  init: RequestInit | undefined,
  selector: (response: Response) => Observable<R>,
): Observable<R>;

Much like the selector passed to multicast, observables composed within the selector will be ‘contained’ within the observable returned by fromFetch and unsubscribing from said observable will facilitate the aborting of any promises returned by Response methods within the selector.

If a selector is not passed, fromFetch should continue to behave as in #4742 and not abort upon unsubscription if the promise returned by fetch has resolved.

Additional Information

Note that the changes introduced in https://github.com/ReactiveX/rxjs/pull/3963/files will ensure that unsubscription from the observable returned by fromFetch will occur ASAP if said observable serves as the source for, say, concatMap. So extending the ‘life’ of the AbortController for an emitted Response - if the selector is not used - would be a non-trivial undertaking.

[curiousercreative]

@cartant thanks for that follow-up. My implementation actually doesn’t have to wait for the next release. In my case, I’m streaming my chunked response using response.body.getReader() and after reading through your article and this MDN reference, I found that I can simply cancel the reader stream when my custom Observable unsubscribes (from a switchMap). Works like a charm, but I wouldn’t have gotten it there without your help!

[cartant]

@curiousercreative BTW, if you are interested, the reason for the selector is explained here: https://ncjamieson.com/understanding-fromfetch/