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

implement create method for mongdb-store #16

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

Conversation

maxcountryman
Copy link
Owner

No description provided.

@maxcountryman
Copy link
Owner Author

Evidently mongo doesn't support transactions unless you use a replica set and it's kind of a pain to set this up for testing.

If any mongo store users read this and would like this to be updated, I would appreciate your input.

@davidbayo10
Copy link

davidbayo10 commented Jul 24, 2024

I think setting up a replica set locally isn't that complicated, as you can have a single-node replica set. With proper documentation, it should be sufficient. I'm really interested in updating these dependencies because my project is using MongoDB as the database and I'm currently experiencing compatibility issues with tower-sessions, tower-sessions-mongodb-store, and MongoDB. What do you think would be a solution to move forward on this as soon as possible? Thank you in advance.

@maxcountryman
Copy link
Owner Author

@davidbayo10 why don't you implement SessionStore yourself?

@davidbayo10
Copy link

Because this package solves all I need. It just needs to be adapted to the new tower-sessions API requirements

@maxcountryman
Copy link
Owner Author

@davidbayo10 it's a small interface which you can adapt directly if you like. It makes no sense to publish a broken implementation as I mentioned yesterday. If you'd like to address that problem then that's fine but if you want to fix your immediate problem I think you have all the tools to do so.

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