Improve error message for memcache session store when dalli isn't loaded #7565

Merged
merged 1 commit into from Sep 7, 2012

Projects

None yet

2 participants

@guilleiguaran
Member

rack/session/dalli is included in dalli gem, no in rack like the old memcache-client session store, then I think is a good idea point to the people to install dalli gem if the file can't be loaded

@guilleiguaran guilleiguaran commented on the diff Sep 7, 2012
...action_dispatch/middleware/session/mem_cache_store.rb
@@ -9,7 +14,6 @@ class MemCacheStore < Rack::Session::Dalli
include SessionObject
def initialize(app, options = {})
- require 'dalli'
@guilleiguaran
guilleiguaran Sep 7, 2012 Member

this is loaded already inside of rack/session/dalli

@carlosantoniodasilva carlosantoniodasilva merged commit e371c66 into rails:master Sep 7, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment