fix: config should set environment before requiring sequelize - Githubissues

rfcx / rfcx-api

Core, Media/Assets and MQTT APIs

https://api.rfcx.org/docs/

Apache License 2.0

0 stars 0 forks source link

fix: config should set environment before requiring sequelize #610

Closed veckatimest closed 6 months ago

veckatimest commented 6 months ago

✅ DoD

[x] Fixes bug when we use env_vars.js (!!)
[ ] API docs updated na
[ ] Release notes updated na
[ ] Deployment notes updated na
[ ] Unit or integration tests added na
[ ] DB migrations tested na

(use na when API docs (Release notes, etc) do not need to be updated)

🛑 Problems

env-vars are set in common/config/index.js, so sequelize must be imported only after config. Because modules are singletons and are created only once