Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix!: use correct case for authsource mongo
Update `authSource` keyword argument to `authsource` Authsource was configurated in camelcase. https://github.com/overhangio/tutor/blob/master/tutor/templates/apps/openedx/settings/partials/common_all.py#L16 Something I think is ok because pymongo uses it in camelcase. https://pymongo.readthedocs.io/en/3.10.1/api/pymongo/mongo_client.html However, the error was presented because edx-platform use another variable when credentials are provided **(LOWERCASE)**. https://github.com/openedx/edx-platform/blob/master/xmodule/mongo_utils.py#L72 Looking more , the `auth_source `is configured with another key name: `authsource` **(LOWERCASE)** from the kwargs. https://github.com/openedx/edx-platform/blob/master/xmodule/mongo_utils.py#L34 This won't change the **kwargs behavior of the camelCase in pymongo `MongoClient` because pymongo has a caseInsentive method. https://github.com/mongodb/mongo-python-driver/blob/3.12.3/pymongo/mongo_client.py#L651 ``it's OK to pass `authsource` instead of authSource as a keyword argument.``
- Loading branch information