PYTHON-5371 - Pass repr(ServerDescription) to logging (#2329)

Author: NoahStapp

pymongo/asynchronous/topology.py

@@ -154,8 +154,8 @@ def __init__(self, topology_settings: TopologySettings):
                 _SDAM_LOGGER,
                 message=_SDAMStatusMessage.TOPOLOGY_CHANGE,
                 topologyId=self._topology_id,
-                previousDescription=initial_td,
-                newDescription=self._description,
+                previousDescription=repr(initial_td),
+                newDescription=repr(self._description),
             )
 
         for seed in topology_settings.seeds:
@@ -514,8 +514,8 @@ async def _process_change(
                 _SDAM_LOGGER,
                 message=_SDAMStatusMessage.TOPOLOGY_CHANGE,
                 topologyId=self._topology_id,
-                previousDescription=td_old,
-                newDescription=self._description,
+                previousDescription=repr(td_old),
+                newDescription=repr(self._description),
             )
 
         # Shutdown SRV polling for unsupported cluster types.
@@ -581,8 +581,8 @@ async def _process_srv_update(self, seedlist: list[tuple[str, Any]]) -> None:
                 _SDAM_LOGGER,
                 message=_SDAMStatusMessage.TOPOLOGY_CHANGE,
                 topologyId=self._topology_id,
-                previousDescription=td_old,
-                newDescription=self._description,
+                previousDescription=repr(td_old),
+                newDescription=repr(self._description),
             )
 
     async def on_srv_update(self, seedlist: list[tuple[str, Any]]) -> None:
@@ -747,8 +747,8 @@ async def close(self) -> None:
                 _SDAM_LOGGER,
                 message=_SDAMStatusMessage.TOPOLOGY_CHANGE,
                 topologyId=self._topology_id,
-                previousDescription=old_td,
-                newDescription=self._description,
+                previousDescription=repr(old_td),
+                newDescription=repr(self._description),
             )
             _debug_log(
                 _SDAM_LOGGER, message=_SDAMStatusMessage.STOP_TOPOLOGY, topologyId=self._topology_id

pymongo/synchronous/topology.py

@@ -154,8 +154,8 @@ def __init__(self, topology_settings: TopologySettings):
                 _SDAM_LOGGER,
                 message=_SDAMStatusMessage.TOPOLOGY_CHANGE,
                 topologyId=self._topology_id,
-                previousDescription=initial_td,
-                newDescription=self._description,
+                previousDescription=repr(initial_td),
+                newDescription=repr(self._description),
             )
 
         for seed in topology_settings.seeds:
@@ -514,8 +514,8 @@ def _process_change(
                 _SDAM_LOGGER,
                 message=_SDAMStatusMessage.TOPOLOGY_CHANGE,
                 topologyId=self._topology_id,
-                previousDescription=td_old,
-                newDescription=self._description,
+                previousDescription=repr(td_old),
+                newDescription=repr(self._description),
             )
 
         # Shutdown SRV polling for unsupported cluster types.
@@ -581,8 +581,8 @@ def _process_srv_update(self, seedlist: list[tuple[str, Any]]) -> None:
                 _SDAM_LOGGER,
                 message=_SDAMStatusMessage.TOPOLOGY_CHANGE,
                 topologyId=self._topology_id,
-                previousDescription=td_old,
-                newDescription=self._description,
+                previousDescription=repr(td_old),
+                newDescription=repr(self._description),
             )
 
     def on_srv_update(self, seedlist: list[tuple[str, Any]]) -> None:
@@ -745,8 +745,8 @@ def close(self) -> None:
                 _SDAM_LOGGER,
                 message=_SDAMStatusMessage.TOPOLOGY_CHANGE,
                 topologyId=self._topology_id,
-                previousDescription=old_td,
-                newDescription=self._description,
+                previousDescription=repr(old_td),
+                newDescription=repr(self._description),
             )
             _debug_log(
                 _SDAM_LOGGER, message=_SDAMStatusMessage.STOP_TOPOLOGY, topologyId=self._topology_id