CodeforSouth / fbi-api

😴 Florida Business Inspections API
http://138.197.90.94/
MIT License
13 stars 5 forks source link

Environmental vars might be set on build, not on run. Audit this. #43

Closed ccjoel closed 7 years ago

ccjoel commented 7 years ago

Setup app with env vars. Build. Push to a different environment server. Run and check env vars. Are they the env vars or the build env, or of the running env?

This might have been introduced in an attempt to log these env vars only once. Used to be the case the these were logged when defined to build, when testing, when starting server, and upon responding to requests- which resulted in a lot of noise in logs.

ccjoel commented 7 years ago

Reproduced. need to remove ^:const in constants.clj

ccjoel commented 7 years ago

merged directly to master. removed ^:const.