#254 remove remote database credentials #255

Sean-Tay · 2017-03-31T16:48:03Z

Phased out the use of config.json for accessing DB parameters, and subsequently removed the usage of currentdb.js - now they all use environment variables - to be set externally, either in the gulp configuration file or in the heroku deployment itself.
Added gulp tasks for running activeStepsConverterScript and fullStepsConverterScript:
'gulp active-convert' or 'gulp full-convert'

…sMongoDB to a single argument: mongoURI

… instead.

…p into username, password, host, port, database name.

# Conflicts: # server/server.js

…, only during gulp test. Updated tests to use the TEST_DB environment variable.

…d DEST db. 2) The converter scripts now export a function, rather than being a runnable file.

…nverter.

2) Removed database credentials from config.json. 3) Made middleware use global database connection.

…T_DB_URI respectively.

Sean-Tay added 19 commits March 31, 2017 22:14

Condensed username, password, host, port, database arguments in acces…

617a5a8

…sMongoDB to a single argument: mongoURI

Changed ModelHandlers initWithParameters method to utilize a mongoURI…

f32cc89

… instead.

Renamed ModelHandler initWithParameters function to initWithUri

b3ace19

Updated config.json to store the full uris rather than splitting it u…

d500148

…p into username, password, host, port, database name.

Changed server to send in one URI instead of five arguments.

869dac0

Merge branch 'develop' into #254_Remove_Remote_Database_Credentials

fb526c3

# Conflicts: # server/server.js

Updated tests to use the new ModelHandler intiWithUri.

1e55b6e

Updated event test.

96b36b3

Updated gulp to fill in the TEST_DB environment variable in the tests…

6a747f9

…, only during gulp test. Updated tests to use the TEST_DB environment variable.

Changed environment variable name from TEST_DB to TEST_DB_URI.

3db284f

Removed placeholder test.

7269e71

Set gulp to pass in the PRODUCTION_DB_URI environment variable.

f1d1546

1) Changed stepsConverterUtil to use environment variables for SRC an…

0be22c5

…d DEST db. 2) The converter scripts now export a function, rather than being a runnable file.

Added gulp tasks for running the activeStepsConverter and fullStepsCo…

8bbbc93

…nverter.

Resolved minor lint issue.

5a65fc7

1) Removed currentdb.js.

30ed001

2) Removed database credentials from config.json. 3) Made middleware use global database connection.

Changed environment variable used to set the production db.

d668fe7

Increased global db pool size to 128.

7af60d7

Renamed environment variable SRC_DB and DEST_DB to SRC_DB_URI and DES…

83e13ac

…T_DB_URI respectively.

Sean-Tay self-assigned this Mar 31, 2017

Sean-Tay requested review from ycewe, Vylantze, Muhammad-Adam and JeanKoh March 31, 2017 16:48

Sean-Tay mentioned this pull request Mar 31, 2017

Remove Remote Database Credentials #254

Closed

Sean-Tay added the database label Apr 1, 2017

Sean-Tay mentioned this pull request Apr 1, 2017

#252 search bugs #257

Merged

ycewe merged commit 5fb36cf into develop Apr 1, 2017

ycewe deleted the #254_Remove_Remote_Database_Credentials branch April 1, 2017 08:21

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#254 remove remote database credentials #255

#254 remove remote database credentials #255

Sean-Tay commented Mar 31, 2017 •

edited

Loading

#254 remove remote database credentials #255

#254 remove remote database credentials #255

Conversation

Sean-Tay commented Mar 31, 2017 • edited Loading

Sean-Tay commented Mar 31, 2017 •

edited

Loading