Don't add None to deprioritized servers

Author: NoahStapp

pymongo/asynchronous/mongo_client.py

@@ -2825,7 +2825,8 @@ async def run(self) -> T:
                     if self._last_error is None:
                         self._last_error = exc
 
-                self._deprioritized_servers.append(self._server)
+                if self._server is not None:
+                    self._deprioritized_servers.append(self._server)
 
     def _is_not_eligible_for_retry(self) -> bool:
         """Checks if the exchange is not eligible for retry"""
@@ -2867,9 +2868,7 @@ async def _get_server(self) -> Server:
             self._session,
             self._operation,
             address=self._address,
-            deprioritized_servers=self._deprioritized_servers
-            if self._deprioritized_servers
-            else None,
+            deprioritized_servers=self._deprioritized_servers,
             operation_id=self._operation_id,
         )
 

pymongo/synchronous/mongo_client.py

@@ -2815,7 +2815,8 @@ def run(self) -> T:
                     if self._last_error is None:
                         self._last_error = exc
 
-                self._deprioritized_servers.append(self._server)
+                if self._server is not None:
+                    self._deprioritized_servers.append(self._server)
 
     def _is_not_eligible_for_retry(self) -> bool:
         """Checks if the exchange is not eligible for retry"""
@@ -2857,9 +2858,7 @@ def _get_server(self) -> Server:
             self._session,
             self._operation,
             address=self._address,
-            deprioritized_servers=self._deprioritized_servers
-            if self._deprioritized_servers
-            else None,
+            deprioritized_servers=self._deprioritized_servers,
             operation_id=self._operation_id,
         )