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

cookie sessions #7

Open
dvv opened this issue Dec 17, 2012 · 9 comments
Open

cookie sessions #7

dvv opened this issue Dec 17, 2012 · 9 comments
Labels

Comments

@dvv
Copy link

dvv commented Dec 17, 2012

Hi!

I recently spawned cookie session projects and wonder would it be feasible to have either inside axiom or as a external helper?

In either event, feedback is welcome.

TIA,
--Vladimir Dronnikov

@tsujigiri
Copy link
Owner

Axiom takes an option for a custom session store module. I like the idea of having your project as some kind of plugin, so people who want to use it just have to put it into their rebar.config and configure the session store module. We would need to write a module, that axiom_session can talk to as a middleman, either in your project or in the axiom source. What do you say?

@dvv
Copy link
Author

dvv commented Dec 17, 2012

Am I right that I need to implement a proxy like https://github.com/tsujigiri/axiom/blob/master/src/axiom_session_ets.erl ?
What is the policy of updating cowboy, as they seem to rewrite cookie handling stuff since 0.6.1, and I rely on that.

@tsujigiri
Copy link
Owner

Regarding the proxy module, that's correct. What do you mean with policy of updating cowboy? When there is a new tagged version I will eventually upgrade the dependency. First I will have to make the http_req record opaque, though, and add some more functions to get to the stuff that's in there. After 0.6.1 the http_req is not exported anymore.

@dvv
Copy link
Author

dvv commented Dec 17, 2012

I see. I'll wait till it gets 0.8.

@rambocoder
Copy link

@tsujigiri regarding including http_req, check out the discussion here ninenines/cowboy#266 and ninenines/cowboy#267

@tsujigiri
Copy link
Owner

@rambocoder I'm aware. Adapting to that shouldn't be too hard. I'm just quite busy atm.

@tsujigiri
Copy link
Owner

I opened a new issue for that: #8

@tsujigiri
Copy link
Owner

I upgraded Cowboy to 0.8.3. You can go ahead now. :)

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

No branches or pull requests

4 participants
@tsujigiri @dvv @rambocoder and others