Require username and password for mongodb by rmunn · Pull Request #1813 · sillsdev/web-languageforge
added
the
engineering
label
May 20, 2024
rmunn
marked this pull request as ready for review
If MONGODB_USER and MONGODB_PASS are set, they will be used to authenticate to the MongoDB server. If they are not set, then a connection request with no authentication will be sent (as per the existing behavior). This allows us to deploy this change, then set MONGODB_USER and MONGODB_PASS later and have those changes picked up without a redeployment. Note that a corresponding change to LfMerge will also be needed.
LfMerge env vars won't be used until we deploy a new build of LfMerge that looks for them, which will be in a PR on the LfMerge repo.
LfMerge now has an alpha build that handles auth. Once we've proved that it works, we'll release a full build of LfMerge and bump this version number again.
Now the check for the `MONGODB_USER` and `MONGODB_PASS` env vars works correctly because it has the right syntax.
This will allow us to change the name of the Mongo database we store our auth in, if in the future we decide not to go with the default name.
rmunn
changed the title
Chore/require password for mongodb
Require username and password for mongodb
rmunn
deleted the
chore/require-password-for-mongodb
branch
rmunn
mentioned this pull request
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters