Skip to content

Commit d456546

Browse files
authored
Merge pull request #1705 from VarmaSANJAY/topic-performance
C50-575 Updating the SSL configuration for accepting username and pas…
2 parents ce0a8b7 + 3623662 commit d456546

File tree

4 files changed

+22
-14
lines changed

4 files changed

+22
-14
lines changed

server/app/app.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -44,22 +44,23 @@ const dboptions = {
4444
host: process.env.DB_HOST || appConfig.db.host,
4545
port: process.env.DB_PORT || appConfig.db.port,
4646
dbName: process.env.DB_NAME || appConfig.db.dbName,
47-
ssl: process.env.DB_SSL || appConfig.db.ssl,
48-
enable_ssl: process.env.ENABLE_SSL || appConfig.db.enable_ssl,
49-
enable_auth: process.env.ENABLE_AUTH || appConfig.db.enable_auth,
47+
ssl: process.env.DB_SSL === 'true' || appConfig.db.ssl,
48+
enable_ssl: process.env.ENABLE_SSL === 'true' || appConfig.db.enable_ssl,
49+
enable_auth: process.env.ENABLE_AUTH === 'true' || appConfig.db.enable_auth,
5050
ssl_config:{
5151
"CAFile": process.env.CAFILE || appConfig.db.ssl_config.CAFile,
5252
"PEMFile": process.env.PEMFILE || appConfig.db.ssl_config.PEMFile
5353
},
5454
auth_config:{
55-
"username":process.env.USERNAME || appConfig.db.auth_config.username,
56-
"password":process.env.PASSWORD || appConfig.db.auth_config.password,
55+
"username":process.env.username || appConfig.db.auth_config.username,
56+
"password":process.env.password || appConfig.db.auth_config.password,
5757
"authenticated":process.env.authenticated || appConfig.db.auth_config.authenticated
5858
}
5959
};
6060

6161
// Initialise the mongodb connections along with that mongoose ORM would be configure
6262
const mongoDbConnect = require('_pr/lib/mongodb');
63+
6364
mongoDbConnect(dboptions, function (err) {
6465
if (err) {
6566
logger.error("Unable to connect to mongo db >>" + err);
@@ -86,7 +87,6 @@ logger.debug('Logger Initialized');
8687

8788

8889
LDAPUser.getLdapUser(function(err, ldapData) {
89-
console.log(JSON.stringify(ldapData))
9090
if (err) {
9191
logger.error("Failed to get ldap-user: ", err);
9292
return;

server/app/model/file-upload/file-upload.js

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,16 +10,16 @@ const dboptions = {
1010
host: process.env.DB_HOST || appConfig.db.host,
1111
port: process.env.DB_PORT || appConfig.db.port,
1212
dbName: process.env.DB_NAME || appConfig.db.dbName,
13-
ssl: process.env.DB_SSL || appConfig.db.ssl,
14-
enable_ssl: process.env.ENABLE_SSL || appConfig.db.enable_ssl,
15-
enable_auth: process.env.ENABLE_AUTH || appConfig.db.enable_auth,
13+
ssl: process.env.DB_SSL === 'true' || appConfig.db.ssl,
14+
enable_ssl: (process.env.ENABLE_SSL === 'true') || appConfig.db.enable_ssl,
15+
enable_auth: process.env.ENABLE_AUTH === 'true' || appConfig.db.enable_auth,
1616
ssl_config:{
1717
"CAFile": process.env.CAFILE || appConfig.db.ssl_config.CAFile,
1818
"PEMFile": process.env.PEMFILE || appConfig.db.ssl_config.PEMFile
1919
},
2020
auth_config:{
21-
"username":process.env.USERNAME || appConfig.db.auth_config.username,
22-
"password":process.env.PASSWORD || appConfig.db.auth_config.password,
21+
"username":process.env.username || appConfig.db.auth_config.username,
22+
"password":process.env.password || appConfig.db.auth_config.password,
2323
"authenticated":process.env.authenticated || appConfig.db.auth_config.authenticated
2424
}
2525
};
@@ -37,12 +37,19 @@ if(dboptions.enable_ssl){
3737
mongoOptions.sslKey = key;
3838
mongoOptions.sslCert = cert;
3939
}
40-
logger.info(connectionString);
40+
41+
if(dboptions.enable_auth){
42+
connectionString = 'mongodb://'+dboptions.auth_config.username+':'+dboptions.auth_config.password+'@' + dboptions.host + ':' + dboptions.port + '/' + dboptions.dbName + '?ssl=' + dboptions.enable_ssl+'&authSource=admin';
43+
}
44+
45+
logger.info("Connecting to mongodb in file upload.");
4146
mongoDbClient.connect(connectionString, {server : mongoOptions}, function (err, db) {
4247
if (err) {
43-
throw "unable to connect to mongodb"
48+
logger.error(err);
49+
throw "unable to connect to mongodb";
4450
return;
4551
}else{
52+
logger.info("Mongodb connected successfully in file upload.");
4653
gfs = Grid(db, mongoDbClient);
4754
}
4855

server/install.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -465,6 +465,7 @@ function getDefaultsConfig() {
465465
port: '27017',
466466
enable_ssl: false,
467467
enable_auth: false,
468+
ssl: false,
468469
ssl_config:{
469470
"CAFile": null,
470471
"PEMFile": null

version.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
{"version":"4.3.0"}
1+
{"version":"4.3.1"}

0 commit comments

Comments
 (0)