fix(analytics,web): sets no-cors on analytics requests for web

[ristewar]

Description

Requests to Google analytics for web aren't setting no-cors so requests are triggering CORS preflight and subsequently failing.

See the following discussion for more details: #7982 (reply in thread)

Related issues

Release Summary

Checklist

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • [x ] Yes
• My change supports the following platforms;
  • Android
  • iOS
• My change includes tests;
  • e2e tests added or updated in packages/\*\*/e2e
  • jest tests added or updated in packages/\*\*/__tests__
• I have updated TypeScript types that are affected by my change.
• This is a breaking change;
  • Yes
  • [ x] No

Test Plan

Tested manually in Chrome. I verified that no CORS preflight is sent and that I received an HTTP 204 response from my Google analytics request.

---

Think react-native-firebase is great? Please consider supporting the project with any of the below:

• 👉 Star this repo on GitHub ⭐️
• 👉 Follow React Native Firebase and Invertase on Twitter

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
react-native-firebase	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 19, 2025 8:51pm

[CLAassistant]

All committers have signed the CLA.

[ristewar]

@mikehardy

[mikehardy]

Thank you for posting this, and really, thanks for digging in and figuring it out. Much appreciated.

Do you think this line should go as it is redundant now?

https://github.com/invertase/react-native-firebase/pull/8238/files#diff-7a131b11aa8aa31bfcc94e8669e61d36253fe15e2da6b813383f6767ca15f603R318

[ristewar]

Yup, done, also retested with that line removed and verified we still get an HTTP 204 from Google Analytics.

[ristewar]

Thank you for posting this, and really, thanks for digging in and figuring it out. Much appreciated.

Do you think this line should go as it is redundant now?

https://github.com/invertase/react-native-firebase/pull/8238/files#diff-7a131b11aa8aa31bfcc94e8669e61d36253fe15e2da6b813383f6767ca15f603R318

Done.

[mikehardy]

A thing of beauty - thanks again

[mikehardy]

It's in the publish queue now, should be out on npmjs.com in a minute