Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consume existing mongo.Db, typo fix and some jsHint fixes #18

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ankitpatial
Copy link

First of all i will thank you to get time and write this package for community.

I was using this package in one of my app, but i wanted to use existing mongo connection(with replicaset settings), but not found the way to do it with current version, so i added in my stuff to handle existing mongo.Db instance. Also i fixed 2 of the typos and did some jsHint cleaning.

I hope these changes can help improve the current version.

Excuse me if that makes no senses to package.

Thanks

@@ -23,7 +23,7 @@ The standard usage, is to just pass an instantiated `MongoStore` instance to the

You can also pass several options to the constructor to tweak your session store:

* db - The name of the db to use, defaults to: `express-sessions`
* db - The name of the db to use, defaults to: `express-sessions` Or you can assign an existing mongo.Db instance
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

README change to highlight the functionality to consume mongo.Db instance

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.

1 participant