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

is session store necessary? #449

Closed
lolgear opened this issue Nov 23, 2015 · 3 comments
Closed

is session store necessary? #449

lolgear opened this issue Nov 23, 2015 · 3 comments

Comments

@lolgear
Copy link

lolgear commented Nov 23, 2015

Is session store necessary for devise_token_auth? It seems that it doesn't need a session ( auth based on tokens ), but if I disable session ( by config.session_store :disabled ), /validate_token works well, but all my controller actions fail.

@lynndylanhurley
Copy link
Owner

Session storage is necessary for OmniAuth to function. Apart from that I don't think it's a requirement. What errors are you seeing?

@lolgear
Copy link
Author

lolgear commented Nov 23, 2015

@lynndylanhurley, I try to migrate from devise to devise_token_auth and I stuck at authentication via safari. Chrome works well, but safari don't. Frontend guy noticed, that I didn't disable session ( session_store was cookies ), and chrome worked well with them, but safari didn't work with them. But when I disable session ( session_store :disabled ), everything became broken and every custom ( my ) action on every controller from api returned an error 401

@zachfeldman
Copy link
Contributor

Hi there @lolgear ,

In an effort to cleanup this project and prioritize a bit, we're marking issues that haven't had any activity in a while with a "close-in-7-days" label. If we don't hear from you in about a week, we'll be closing this issue. Obviously feel free to re-open it at any time if it's the right time or this was done in error!

If you are still having the issue (especially if it's a bug report) please refer to our new Issue Template to provide some more details to help us solve it.

Hope all is well.

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

3 participants