Capture failed HTTP requests by default

[markushi]

📜 Description

💡 Motivation and Context

Fixes #2767

💚 How did you test it?

📝 Checklist

• I reviewed the submitted code.
• I added tests to verify the changes.
• No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
• I updated the docs if needed: chore(android): Mention sdk 7.0.0 will capture failed HTTP requests by default sentry-docs#7238
• Review from the native team if needed.
• No breaking change or entry added to the changelog.
• No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

[github-actions]

	Messages
📖	Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 5a214bd

[github-actions]

Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	330.94 ms	359.27 ms	28.33 ms
Size	1.72 MiB	2.28 MiB	572.21 KiB

Previous results on branch: feat/default-capture-failed-requests

Startup times

Revision	Plain	With Sentry	Diff
41de78f	310.19 ms	370.77 ms	60.58 ms

App size

Revision	Plain	With Sentry	Diff
41de78f	1.72 MiB	2.28 MiB	572.22 KiB

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (feat/7.0.0@0566579). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files

@@              Coverage Diff              @@
##             feat/7.0.0    #2794   +/-   ##
=============================================
  Coverage              ?   81.16%           
  Complexity            ?     4496           
=============================================
  Files                 ?      348           
  Lines                 ?    16670           
  Branches              ?     2267           
=============================================
  Hits                  ?    13530           
  Misses                ?     2193           
  Partials              ?      947           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[romtsn]

Btw, I think we should do this for apollo-3 as well? @marandaneto what's your take? or is it still too early for enabling it by default?

[marandaneto]

Btw, I think we should do this for apollo-3 as well? @marandaneto what's your take? or is it still too early for enabling it by default?

That we can't because GraphQL errors include request/response by default, it should be opt-in.