core: audit abort listeners and bubble up `reason`

[Peng Xu]

if (event.type === "abort") {
// add reason here? abortController.abort("some reason");
abortController.abort();
}

[Maor Leger]

Hiya @xupeng543210 - thanks for reaching out! That's a good question. We certainly can add a reason; however, I am not sure we have any relevant reason to add - we could add a generic message but I don't think the Event payload we get to the abortListener defined in fetchHttpClient provides any meaningful information according to the docs

We may be able to fetch the abortSignal of the request and pass along its reason, but I don't know if that would be helpful / accurate and will need to investigate a bit.

If we were to add a generic message here, what information would you find valuable? We may be able to do better and would love your input!

[Peng Xu]

uh, @maorleger , I found there is a reason passed in the event which is the user passed in, it should be accurate I think. Hope this is helpful

[Maor Leger]

uh, @maorleger , I found there is a reason passed in the event which is the user passed in, it should be accurate I think. Hope this is helpful

Yeah that's super helpful - thank you! This should be doable and will improve the experience for sure, appreciate the suggestion!

Just to set expectations, I plan to tackle this in April and ship it in May hopefully. Thanks!