v2 backport: Ignore OPTIONS requests in Oauth server

[RamIdeas]

What this PR solves / how to test

This PR backports the fix (for wrangler login to work with Chrome v123) to wrangler v2. This patch has already been applied to wrangler v3 in #5324.

Fixes #5328

Author has addressed the following

• Tests
  • TODO (before merge)
  • Included
  • Not necessary because:
• Changeset (Changeset guidelines)
  • TODO (before merge)
  • Included
  • Not necessary because:
• Public documentation
  • TODO (before merge)
  • Cloudflare docs PR(s): https://github.com/cloudflare/cloudflare-docs/pull/...
  • Not necessary because: bugfix

[changeset-bot]

🦋 Changeset detected

Latest commit: e8d9aa8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
wrangler	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (v2-maintenance@da90e6a). Click here to learn what that means.

Additional details and impacted files

@@                Coverage Diff                @@
##             v2-maintenance    #5337   +/-   ##
=================================================
  Coverage                  ?   74.53%           
=================================================
  Files                     ?      179           
  Lines                     ?    10941           
  Branches                  ?     2912           
=================================================
  Hits                      ?     8155           
  Misses                    ?     2786           
  Partials                  ?        0           

Files	Coverage Δ
packages/wrangler/src/user/user.ts	72.38% <100.00%> (ø)

[petebacondarwin]

I know this is a backport of the other changeset, but as I commented there, I think it would be helpful to include information about symptoms of the problem that the user would have seen and the fact that this was needed to fix a change in Chrome, mentioning the version etc, so that it makes it easier for users to identify this release as solving that problem.

[petebacondarwin]

It might be worth just calling out the error messages that people might see:

Expired! Auth code or refresh token needs to be renewed.

and

ErrorInvalidGrant

[petebacondarwin]

Thanks for taking this on @RamIdeas