Skip to content

Add authSource & ssl options to MongoDB connector#753

Merged
lauriejim merged 8 commits into
strapi:masterfrom
abdonrd:mongodb-authsource-ssl
Mar 14, 2018
Merged

Add authSource & ssl options to MongoDB connector#753
lauriejim merged 8 commits into
strapi:masterfrom
abdonrd:mongodb-authsource-ssl

Conversation

@abdonrd
Copy link
Copy Markdown
Contributor

@abdonrd abdonrd commented Mar 8, 2018

Based on #738.
Fixes #694.

I have not tried it yet.

@lauriejim
Copy link
Copy Markdown
Contributor

lauriejim commented Mar 12, 2018

Can you test it please and tell me if it fix your issue ?
After that I will be able to merge it ;)

@abdonrd
Copy link
Copy Markdown
Contributor Author

abdonrd commented Mar 12, 2018

I'll try it as soon as I can! (tomorrow I guess)

@lauriejim
Copy link
Copy Markdown
Contributor

Perfect ! Thank you

@abdonrd
Copy link
Copy Markdown
Contributor Author

abdonrd commented Mar 13, 2018

@lauriejim it works for me with using the config/environments/development/database.json! 🎉

But it doesn't work with the interactive CLI from strapi new myDevelopmentProject --dev.

@lauriejim lauriejim merged commit 5d6968a into strapi:master Mar 14, 2018
@abdonrd abdonrd deleted the mongodb-authsource-ssl branch March 14, 2018 09:15
@abdonrd
Copy link
Copy Markdown
Contributor Author

abdonrd commented Mar 14, 2018

@lauriejim remember that it doesn't work with the interactive CLI from strapi new myDevelopmentProject --dev, only with the config files.

screen shot 2018-03-14 at 09 32 16

With the same data in the config files, it works.

@lauriejim
Copy link
Copy Markdown
Contributor

It's normal. Strapi try a npm install strapi-mongoose@alpha and your update is not publish to npm.
If you update the line where the test connection folder is created by npm link strapi-mongoose it should work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants