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

Don't step over :session_id setting on admin apps #1443

Merged
merged 1 commit into from Oct 13, 2013

Conversation

Projects
None yet
2 participants
@dariocravero
Contributor

dariocravero commented Oct 10, 2013

The :session_id is currently overwritten in the admin regardless whether it was set globally or not. This doesn't allow for shared sessions between admins and apps. This patch fixes this issue. @padrino/core-members thoughts?

ujifgc added a commit that referenced this pull request Oct 13, 2013

Merge pull request #1443 from padrino/dont_override_session_id_admin_…
…if_set

Don't step over :session_id setting on admin apps

@ujifgc ujifgc merged commit 8a8cb19 into master Oct 13, 2013

1 check passed

default The Travis CI build passed
Details

@ujifgc ujifgc deleted the dont_override_session_id_admin_if_set branch Oct 13, 2013

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