Skip to content

Conversation

matthewdale
Copy link
Contributor

DRIVERS-2672

Please complete the following before merging:

  • Update changelog.
  • Make sure there are generated JSON files from the YAML test files.
  • Test changes in at least one language driver.
  • Test these changes against all server versions and topologies (including standalone, replica set, sharded clusters, and serverless).

@matthewdale matthewdale force-pushed the drivers2672-oidc-spec-tests branch 3 times, most recently from 4ef1872 to bc78b17 Compare October 16, 2023 23:33
@matthewdale matthewdale force-pushed the drivers2672-oidc-spec-tests branch from bc78b17 to fff9888 Compare October 16, 2023 23:46
is enabled. If false, tests MUST NOT run if authentication is enabled. If this
field is omitted, there is no authentication requirement.

- ``authMechanism``: Optional string. Specifies an authentication mechanism that
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How are we expected to determine this? Wire version? saslSupportedMechs?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

db.adminCommand({'getParameter':"1", "authenticationMechanisms": "1"}). In PyMongo we actually already retrieved and stored all of the server parameters, so we can just look up authenticationMechanisms in that dict

@matthewdale
Copy link
Contributor Author

These changes are now part of #1471. Closing this PR.

@matthewdale matthewdale closed this Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants