Skip to content

Commit

Permalink
Merge pull request #2 from SnehaGunda/SnehaGunda-CDB
Browse files Browse the repository at this point in the history 
Updating code as per GitHub issue
  • Loading branch information
@burkeholland
burkeholland committed Apr 12, 2018
2 parents 2883063 + 7604ba1 commit b881f3e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions server/mongo.js
View file
Expand Up @@ -3,10 +3,10 @@ const env = require('./env/environment');

mongoose.Promise = global.Promise;

const mongoUri = `mongodb://${env.dbName}:${env.key}@${env.dbName}.documents.azure.com:${env.cosmosPort}/?ssl=true`;
const mongoUri = `mongodb://${env.dbName}.documents.azure.com:${env.cosmosPort}/?ssl/=true`;

This comment has been minimized.

Sign in to view

Copy link
@ydogandjiev

ydogandjiev Apr 13, 2018

@burkeholland, @SnehaGunda, the "/" added to "?ssl/=true" is actually unnecessary and causes connection failures. I am also encountering some additional issues with the latest react-cosmosdb app but I can't quite figure out why. The sample I built based on Burke's tutorial does work thou:
https://github.com/ydogandjiev/express-react-starter

This comment has been minimized.

Sign in to view

Copy link
@Stevitenk

Stevitenk Jun 4, 2019

Yo también tengo ese tipo de problemas

function connect() {
return mongoose.connect(mongoUri, { useMongoClient: true });
return mongoose.connect(mongoUri, { auth: { user: env.dbName, password: env.key }});
}

module.exports = {
Expand Down

0 comments on commit b881f3e

Please sign in to comment.