fix handling of 'none'

Author: blink1073

pymongo/asynchronous/auth.py

@@ -180,7 +180,7 @@ def _auth_key(nonce: str, username: str, password: str) -> str:
 def _canonicalize_hostname(hostname: str, option: str | bool) -> str:
     """Canonicalize hostname following MIT-krb5 behavior."""
     # https://github.com/krb5/krb5/blob/d406afa363554097ac48646a29249c04f498c88e/src/util/k5test.py#L505-L520
-    if option in [False, None]:
+    if option in [False, "none"]:
         return hostname
 
     af, socktype, proto, canonname, sockaddr = socket.getaddrinfo(

pymongo/auth_shared.py

@@ -89,8 +89,6 @@ def __hash__(self) -> int:
 
 def _validate_canonicalize_host_name(value: str | bool) -> str | bool:
     valid_names = [False, True, "none", "forward", "forwardAndReverse"]
-    if value == "none":
-        value = False
     if value in ["true", "false", True, False]:
         return value in ["true", True]
 

pymongo/synchronous/auth.py

@@ -177,7 +177,7 @@ def _auth_key(nonce: str, username: str, password: str) -> str:
 def _canonicalize_hostname(hostname: str, option: str | bool) -> str:
     """Canonicalize hostname following MIT-krb5 behavior."""
     # https://github.com/krb5/krb5/blob/d406afa363554097ac48646a29249c04f498c88e/src/util/k5test.py#L505-L520
-    if option in [False, None]:
+    if option in [False, "none"]:
         return hostname
 
     af, socktype, proto, canonname, sockaddr = socket.getaddrinfo(