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

Session management check failed #48

Open
intinig opened this issue Mar 26, 2014 · 1 comment
Open

Session management check failed #48

intinig opened this issue Mar 26, 2014 · 1 comment

Comments

@intinig
Copy link

intinig commented Mar 26, 2014

I am using redis-store for sessions, but I still get this error:

18:26:12 [!] dawn: Owasp Ror CheatSheet: Session management check failed
18:26:12 [$] dawn: Description: By default, Ruby on Rails uses a Cookie based session store. What that means is that unless you change something, the session will not expire on the server. That means that some default applications may be vulnerable to replay attacks. It also means that sensitive information should never be put in the session.
18:26:12 [$] dawn: Solution: Use ActiveRecord or the ORM you love most to handle your code session_store. Add "Application.config.session_store :active_record_store" to your session_store.rb file.
18:26:12 [$] dawn: Evidence:
18:26:12 [$] dawn:  In your session_store.rb file you are not using ActiveRercord to store session data. This will let rails to use a cookie based session and it can expose your web application to a session replay attack.
18:26:12 [$] dawn:  {:filename=>"/Users/intinig/src/rgts/config/initializers/session_store.rb", :matches=>[]}
@thesp0nge thesp0nge added this to the Version 2.0.0 (Finn McMissile) milestone Mar 27, 2014
@thesp0nge
Copy link
Owner

This is an issue #41 duplicate.
I'm going to improve this check in next major

@thesp0nge thesp0nge self-assigned this Mar 27, 2014
@thesp0nge thesp0nge removed this from the v2-0-0 milestone Apr 14, 2023
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

2 participants