remove X-Requested-With: XMLHttpRequest header

[lion7]

Description

This PR effectively reverts the changes of PR #4934 (or more specific the commit 937c8f6).

Motivation and Context

As described in more detail in issue #6081, the use of the X-Requested-With header triggers a preflight on most authentication backends. Most authentication endpoints do not implement CORS, and when they do the header X-Requested-With is not in the list of allowed headers.

Note that the motivation of the original PR that introduced this change seems to be debatable. The popup shown in the screenshots there seem to be caused by a backend sending a WWW-Authenticate header, which is kind of weird.

Fixes #6081

How Has This Been Tested?

Running the latest swagger-ui with Dex IDP (https://dexidp.io/). My browser fails to fetch the token after going through the authorization code flow. Replacing swagger-ui with a local build of this PR results in successful authentication.

Screenshots (if appropriate):

Checklist

My PR contains...

• No code changes (src/ is unmodified: changes to documentation, CI, metadata, etc.)
• Dependency changes (any modification to dependencies in package.json)
• Bug fixes (non-breaking change which fixes an issue)
• Improvements (misc. changes to existing features)
• Features (non-breaking change which adds functionality)

My changes...

• are breaking changes to a public API (config options, System API, major UI change, etc).
• are breaking changes to a private API (Redux, component props, utility functions, etc.).
• are breaking changes to a developer API (npm script behavior changes, new dev system dependencies, etc).
• are not breaking changes.

Documentation

• My changes do not require a change to the project documentation.
• My changes require a change to the project documentation.
• If yes to above: I have updated the documentation accordingly.

Automated tests

• My changes can not or do not need to be tested.
• My changes can and should be tested by unit and/or integration tests.
• If yes to above: I have added tests to cover my changes.
• If yes to above: I have taken care to cover edge cases in my tests.
• [?] All new and existing tests passed.

[mosinve]

Is it still supposed to merge?

[syl20bnr]

Well that would be cool to merge this since it is still a blocker since 2024.