-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
port over http cookie layer logic #44
Comments
It probably also makes sense to take https://www.lpalmieri.com/posts/biscotti-http-cookies-in-rust/ into account and see if there are lessons that can be applied from there in this porting work. |
Hi @GlenDC I would like to take it.I am confusing about where to start it. A cookie manager in rama::http::layer::cookie? Thanks for your helping. |
Hi sure, do know that this issue is a lot more involved and will take quit some work and time to get this one finished. Wouldn't be surprised if it takes 3 full work days up to a week, depending how fast you are. So it's definitely not a small one by any chance. If you want something smaller or easier (for now), these are so far still available as well: That said if you do want to take it, it is possible,
For now not sure how it would all fit together or be implemented. I was holding off the design for this for 0.3, and as such it's low prio. If you think you need it earlier, or feel motivated and have the skills to push it forward, do tell me and you can take it. Otherwise I would suggest to go for another issue first. |
Actually thinking about it. Most like likely we'll want to fork
This is really a big task, to do on top of the integration as layers and services, and examples. I think you might want to take up another task, as I feel this is too much to ask. Do tell me if you think otherwise though. Don't want to speak for you either. |
This story is about porting over https://docs.rs/tower-cookies/latest/tower_cookies/ and make it available under
rama::http::layer::cookie
(a new module) => https://github.com/plabayo/rama/tree/main/src/http/layer.Ready for others to pick up as there is no immediate plans to work on it. Feel free to ask questions here or on discord, or ask for help or guidance. I am here for you. Once we are working towards the v0.3 milestone I might pick it up at some point myself if no one volunteered by then.
The text was updated successfully, but these errors were encountered: