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

Remove the tmp/sessions folder and its clear task #18314

Merged
merged 1 commit into from Jan 3, 2015

Conversation

Projects
None yet
2 participants
@robin850
Member

robin850 commented Jan 3, 2015

Hello,

Commit 1aea470 introduced the tmp/sessions directory but this was at a time when the default way to store sessions was on the file system under the tmp directory. It looks like this directory is not needed anymore so let's get rid of it.

Just for the record, this has been spotted by yui-knk in #17564 (comment).

Have a nice day.

Remove the tmp/sessions folder and its clear task
Commit 1aea470 introduced this directory but this was at a time when the
default way to store sessions was on the file system under the tmp
directory.

Let's remove references to it from the documentation as well.

[Robin Dupret & yui-knk]

@robin850 robin850 added the railties label Jan 3, 2015

rafaelfranca added a commit that referenced this pull request Jan 3, 2015

Merge pull request #18314 from robin850/rm-tmp-sessions
Remove the tmp/sessions folder and its clear task

@rafaelfranca rafaelfranca merged commit c2094f5 into rails:master Jan 3, 2015

1 check was pending

continuous-integration/travis-ci The Travis CI build is in progress
Details

@robin850 robin850 deleted the robin850:rm-tmp-sessions branch Jan 3, 2015

prathamesh-sonpatki added a commit to prathamesh-sonpatki/rails that referenced this pull request Jun 11, 2016

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