fix(threading): Handle channels shadowing

[sentrivana]

Description

Folks might have a channels module that has nothing to do with the channels package we're expecting. This will then make the code throw an AttributeError which we don't handle.

Issues

• ClosesAttributeError: module 'channels' has no attribute '__version__' #5298

Reminders

• Please add tests to validate your changes, and lint your code using tox -e linters.
• Add GH Issue ID & Linear ID (if applicable)
• PR title should use conventional commit style (feat:, fix:, ref:, meta:)
• For external contributors: CONTRIBUTING.md, Sentry SDK development docs, Discord community

[alexander-alderman-webb]

💯

[javiertejero]

this implies that if there is an AttributeError because of channels line, django_version will be set to None and that is not ideal... perhaps we should split this try/except in 2 parts, 1 for django and another for channels?

[javiertejero]

this implies that if there is an AttributeError because of channels line, django_version will be set to None and that is not ideal... perhaps we should split this try/except in 2 parts, 1 for django and another for channels?

[sentrivana]

That's ok, django_version and channels_version both are only used for warning about a very specific Django/channels/sentry-sdk incompatibility that only appears in specific versions.

But you raise a good point in that it's not clear what the variable is used for. It might happen that someone extends the logic at some point using django_version for some other purpose, not realizing it's not always populated correctly. So it's worth making this nicer.

[sentrivana]

d34f796

[github-actions]

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

---

Bug Fixes 🐛

• fix(clickhouse): Guard against module shadowing by alexander-alderman-webb in #5250
• fix(gql): Revert signature change of patched gql.Client.execute by alexander-alderman-webb in #5289
• fix(litellm): Guard against module shadowing by alexander-alderman-webb in #5249
• fix(pure-eval): Guard against module shadowing by alexander-alderman-webb in #5252
• fix(ray): Guard against module shadowing by alexander-alderman-webb in #5254

• fix(threading): Handle channels shadowing by sentrivana in #5299

• fix(typer): Guard against module shadowing by alexander-alderman-webb in #5253

Documentation 📚

• docs: Update Python versions banner in README by sentrivana in #5287

Internal Changes 🔧

• ci(release): Switch from action-prepare-release to Craft by BYK in #5290

---

_{🤖 This preview updates automatically when you update the PR.}