search

cds-snc / pulse

Archived: [Project has been split out into two components, @ https://github.com/cds-snc/tracker and https://github.com/cds-snc/track-web ] Check whether a Government of Canada domain is adhering to best security practices.
Other
6 stars 1 forks source link

add missing default db name #106

Closed buckley-w-david closed 6 years ago

buckley-w-david commented 6 years ago

This PR changes the default DB name to match the default of other places. This was missed in the rebranding PR.

buckley-w-david commented 6 years ago

Actual name is track as well. It's actually a bit annoying since due to how the flask-pymongo extension is written the database name it uses by default is the name of the flask package if one isn't defined in the connection string.

Since the flask package is now called track, so is the DB. We could explicitly use another DB, but that would require either

  1. Adding some extra code to switch what DB the connection is pointed at based on config options or
  2. Edit the connection string to specify a default DB

Neither are particularly hard (I'd probably go for the first one since it's cleaner and easier to bake into the automation), just haven't bothered to as of yet.