[ruby-dev:13058] Re: [bug?] CGI::Session::MemoryStore (ruby-1.6.x, ruby-1.7.x)

まつもと ゆきひろです

In message "[ruby-dev:13053] [bug?] CGI::Session::MemoryStore (ruby-1.6.x, ruby-1.7.x)"
    on 01/04/29, akira yamada / やまだあきら <[e-mail]> writes:

|バグなのかどうなのかよくわからないのですが, 
|CGI::Session::MemoryStore#initializeで
|無条件にGLOBAL_HASH_TABLEに{}を入れてしまっているのは
|まずいように思います. 

	GLOBAL_HASH_TABLE[@session_id] ||= {}

の方が良いですかねえ。そんな気がしてきました。

|あとGLOBAL_HASH_TABLEにアクセスしているところを
|Mutex#synchronize{}でくくってやらないとまずいようにも思います. 

現状ではそれぞれがatomicな処理なので問題ないです。||= に変更
すると厳密には必要になりますね。ただ、MemoryStoreはそれでな
くても使い物にはならないので、あんまりやる気がないですね。
そのために thread.rb をロードするのも無駄な気がしますし。

|# CGI::Session::MemoryStore#deleteの中では
|# Hash#deleteを呼ぶべきですかね?

そうです。忘れてました。

                                まつもと ゆきひろ /:|)