chore: More logging to stripe update default method

[RulaKhaled]

Purpose/Motivation

What is the feature? Why is this being done?
Need to debug: https://codecov.sentry.io/issues/5909837528/?environment=production&project=5215654&query=is%3Aunresolved%20issue.priority%3A%5Bhigh%2C%20medium%5D&referrer=issue-stream&sort=date&stream_index=3
we're not getting enough logs in prod

What does this PR do?

Include a brief description of the changes in this PR. Bullet points are your friend.

Notes to Reviewer

Anything to note to the team? Any tips on how to review, or where to start?

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

[seer-by-sentry]

🚨 Sentry detected 4 potential issues in your recent changes 🚨

• Line 732, services/billing.py: Item "None" of "InvoiceSettings | None" has no attribute "default_payment_method"

  • customer.invoice_settings can be None, leading to AttributeError.

Lower risk findings

• Line 764, services/billing.py: Argument 1 to "modify" of "PaymentMethod" has incompatible type "str | Any | PaymentMethod"; expected "str"

  • default_payment_method might not be a string, causing a type error.

• Line 780, services/billing.py: Do not catch blind exception: 'Exception'

  • Catching Exception is too broad; catch specific exceptions.

• Line 781, services/billing.py: Use 'logging.exception' instead of 'logging.error'

  • Use logging.exception to include traceback in logs.

_{Did you find this useful? React with a 👍 or 👎}

[joseph-sentry]

it might be good to move the dynamic data in the log message to the extra argument and store the relevant info in the same dict that you pass to each log so they all consistently contain the same data

[codecov-notifications]

Codecov Report

Attention: Patch coverage is 66.66667% with 4 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
services/billing.py	66.66%	4 Missing ⚠️

📢 Thoughts on this report? Let us know!

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 4 lines in your changes missing coverage. Please review.

Project coverage is 96.33%. Comparing base (c2edbb3) to head (76aa30c).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
services/billing.py	66.66%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1288      +/-   ##
==========================================
- Coverage   96.35%   96.33%   -0.03%     
==========================================
  Files         488      488              
  Lines       16919    16929      +10     
==========================================
+ Hits        16302    16308       +6     
- Misses        617      621       +4     

Flag	Coverage Δ
unit	96.33% <66.66%> (-0.03%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.