Memory session not functional ...

Dear raid

You need to describe your errors more closely. Please read http://www.chiark.greenend.org.uk/~sgtatham/bugs.html (among others) and restate your bugreport.

Until then, I invalidate this ticket. By that I am not saying your error is invalid, it might be very real and I thank you for reporting it, but the report per se is not up to par. Please appreciate that precise reporting is needed for this (potential) bug to be removed. When you redescribe the bug, just reopen the ticket.

All the best. Jon Egil Strand

ahhh You to lazy to verify ? what more can i say then memory sessions do not work!
Please try play with :
#:session_store: cookie

:session_store: memory
while using this helper : http://pastie.caboo.se/97914

then set for ex. session[:asession] = "true" and when running in memory mode it is clerely broken, then try run the same with cookie and alas it is functional.

dear raid

My agenda is to encourage a decent level of bug reporting. Thank you for elaborating.

All the best JE

Hi all, I use this ticket because I too had problems with session_store set to "memory".

I'm using merb svn and after checking the code, I noticed two things:

First:

in lib/merb/controller.rb at line 12 we have:

self._session_id_key = :_session_id

should be:

self._session_id_key = "_session_id"

The reason is that set_cookie (in lib/mixins/controller.rb at line 184) is using to_s on the name of the cookie.

Second:

The class MemorySession? is using the class MemorySessionContainer? to keep track of all the sessions.

After checking MemorySessionContainer?, the method 'setup' defined in lib/session/memory_session.rb at line 96) is never used.

For my testing I just put this *ugly* code in the persist method (same file at line 42):

MemorySessionContainer?.setup if MemorySessionContainer?.sessions.nil?

After I restart the server, it all worked well :)

There must be a better solution for the second part though...

Keep on the good work guys [[BR]]

Alex

(In [650]) fixed memory sessions. closes #182