Investigate AbortController() pre-emptively stopping requests

[MichaelSwigerAtBentley]

In Desktop View Test in iTwin/viewer when attempting to get imodels for a given iTwin, the api call is stalling and being aborted:

This causes the iModels page to never load, leaving 3 loading animations cycling forever, as described in this Issue

Removing the
signal: abortController.signal
from useIModelData.ts#L127 corrects the issue.

Removing React.StrictMode from the app also corrects the issue.

We should investigate why this is interfering with the request.

Additional info:
Running the request with the same accessToken and headers does return the iModels as normal, ruling out a clientId or auth issue.

[MichaelSwigerAtBentley]

@aruniverse - @hl662 and I were able to reproduce the user's issue, and it seems related to the abortController as mentioned above.

[aruniverse]

is this an issue with react 18 strict mode?

[MichaelSwigerAtBentley]

is this an issue with react 18 strict mode?

Yes, that did it. Removing the strict mode corrects the issue. Should I remove it from the desktop-viewer-test?

[aruniverse]

is this an issue with react 18 strict mode?

Yes, that did it. Removing the strict mode corrects the issue. Should I remove it from the desktop-test-app?

Remove it from the template and suggest that as a fix to the user

[MichaelSwigerAtBentley]

Closing this issue with PR: iTwin/viewer#339

[aruniverse]

Re-opening, as there still is an issue with this custom hook in react 18 strict mode