ref(utils): Explicitly use None default when checking metadata (#4039)

Fixes #4035

As described in the above issue, starting in Python 3.14
importlib_metadata 8 provides the desired behavior, raising KeyError on
a missing key. In preparation for this change, and to remove a
DeprecationWarning, we should explicitly default to None when getting
metadata.

Author: mpurnell1

sentry_sdk/utils.py

@@ -1665,7 +1665,7 @@ def _generate_installed_modules():
 
         yielded = set()
         for dist in metadata.distributions():
-            name = dist.metadata["Name"]
+            name = dist.metadata.get("Name", None)  # type: ignore[attr-defined]
             # `metadata` values may be `None`, see:
             # https://github.com/python/cpython/issues/91216
             # and

tests/test_utils.py

@@ -650,12 +650,12 @@ def test_installed_modules():
 
     if importlib_available:
         importlib_distributions = {
-            _normalize_distribution_name(dist.metadata["Name"]): version(
-                dist.metadata["Name"]
+            _normalize_distribution_name(dist.metadata.get("Name", None)): version(
+                dist.metadata.get("Name", None)
             )
             for dist in distributions()
-            if dist.metadata["Name"] is not None
-            and version(dist.metadata["Name"]) is not None
+            if dist.metadata.get("Name", None) is not None
+            and version(dist.metadata.get("Name", None)) is not None
         }
         assert installed_distributions == importlib_distributions