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

Create Redis session backend #69

Open
bradleybeddoes opened this issue Dec 5, 2017 · 4 comments
Open

Create Redis session backend #69

bradleybeddoes opened this issue Dec 5, 2017 · 4 comments

Comments

@bradleybeddoes
Copy link
Contributor

We currently have an in memory session backend which is very useful for proving out basic use cases and initial deployments but isn't suitable for larger deployments i.e. with multiple containers that we're treating like 🐄.

Create a second reference implementation (standalone crate) that utilises Redis (and hence services such as elasticache) as a session Backend .

It would be a good idea to reach out to @benashford to find out the status of https://github.com/benashford/redis-async-rs noting no recent commits. There is another crate in a similar state with no recent commits at https://github.com/tokio-rs/tokio-redis which would also be worth finding out status on.

A non Tokio integrated, but well maintained crate exists at https://github.com/mitsuhiko/redis-rs

@benashford
Copy link

Regarding redis-async-rs, it's still currently active, there's a feature branch where most of the recent commits have been going, but due to other time pressures it's going to be a very slow moving project.

Of course, you're more than welcome to give it a try and see if it fits your needs, but it is way short of the level of features (and testing) that redis-rs has at this stage.

@bradleybeddoes
Copy link
Contributor Author

Thanks for the info @benashford will poke around your feature branch when I have some time.

@Marwes
Copy link

Marwes commented Mar 1, 2018

For https://github.com/mitsuhiko/redis-rs I made a pair of PRs to add async support for it. It is still in queue while I finish a release for the parser dependendcy that they need but that should be done in ~2 weeks.

redis-rs/redis-rs#141 redis-rs/redis-rs#143

@bradleybeddoes
Copy link
Contributor Author

@Marwes this is fantastic. Could you keep us updated on progress by chance?

Once this is merged trying it out with Gotham would be an interesting exercise.

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

No branches or pull requests

4 participants