Address review feedback

aclark4life · aclark4life · commit 9314d696f9ce · 2026-03-17T15:59:10.000-04:00
diff --git a/doc/changelog.rst b/doc/changelog.rst
@@ -6,15 +6,6 @@ Changes in Version 4.17.0 (2026/XX/XX)
 
 PyMongo 4.17 brings a number of changes including:
 
-- Remove unused validation functions from pymongo/common.py
-
-  - validate_positive_integer_or_none (line 236) — superseded by
-    validate_non_negative_integer_or_none which is used
-  - validate_int_or_basestring (line 264) — a variant of
-    validate_non_negative_int_or_basestring which is used
-  - validate_auth_option (line 823) — validates auth mechanism properties but
-    is never invoked
-
 - Added the :meth:`~pymongo.asynchronous.client_session.AsyncClientSession.bind` and :meth:`~pymongo.client_session.ClientSession.bind` methods
   that allow users to bind a session to all database operations within the scope of a context manager instead of having to explicitly pass the session to each individual operation.
   See <PLACEHOLDER> for examples and more information.
diff --git a/pymongo/common.py b/pymongo/common.py
@@ -796,8 +796,6 @@ def validate_server_monitoring_mode(option: str, value: str) -> str:
     "waitqueuetimeoutms",
 ]
 
-_AUTH_OPTIONS = frozenset(["authmechanismproperties"])
-
 
 def _get_validator(
     key: str, validators: dict[str, Callable[[Any, Any], Any]], normed_key: Optional[str] = None

Original file line number	Diff line number	Diff line change
`@@ -796,8 +796,6 @@ def validate_server_monitoring_mode(option: str, value: str) -> str:`
`796`	`796`	`"waitqueuetimeoutms",`
`797`	`797`	`]`
`798`	`798`
`799`		`-_AUTH_OPTIONS = frozenset(["authmechanismproperties"])`
`800`		`-`
`801`	`799`
`802`	`800`	`def _get_validator(`
`803`	`801`	`key: str, validators: dict[str, Callable[[Any, Any], Any]], normed_key: Optional[str] = None`