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

save session fix #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zakinat
Copy link

@zakinat zakinat commented Feb 25, 2021

i couldn't able to save the sessions using "connect-mongo" , so i found another way using "connect-mongodb-session".
so i added the dependence "connect-mongodb-session" to" package.json" and imported it correctly in "app.js".
P.S (it is my first contribution in github like that and i hope i done every thing just right and my update is helpful for other
thanks allot you are helping me a lot in the youtube "

@drsimplegraffiti
Copy link

Thanks zakinat...it worked

Copy link

@FinzyPHINZY FinzyPHINZY left a comment

Choose a reason for hiding this comment

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

Thanks a lot for your contribution. This is great and really helpful but you don't need to install another package. simply delete the session call after the require statement and update the store value in the session middleware to the below code.

store: MongoStore.create({ mongoUrl: process.env.MONGO_URI }),

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.

3 participants