dango 0.4.5 → 0.4.6

data/lib/dango/server_framework.rb

@@ -72,10 +72,12 @@ class DangoServerFramework
   
   def exit_process() # プロセス終了処理
     debug_print("shutdown")
-    begin
-#      File.delete(@pid_file)
-    rescue 
-#      puts "failed delete file. #{@pid_file}:#{$!.class}:#{$!.message}"
+    if @is_pid_file_create
+      begin
+        File.delete(@pid_file)
+      rescue 
+        puts "failed delete file. #{@pid_file}:#{$!.class}:#{$!.message}"
+      end
     end
     puts "shutdown"
     exit!
@@ -87,6 +89,8 @@ class DangoServerFramework
     @gserver = nil # Gserver用の変数
     @server_reload = nil # サーバーのリロード用フラグ
     
+    @is_pid_file_create = nil # pidファイルを作ったかどうかのフラグ
+    
     # SIGINT の捕捉
     Signal.trap(:INT) do 
       logger.warn "SIGINT\n#{caller(0).pretty_inspect}"
@@ -183,6 +187,7 @@ class DangoServerFramework
       
       GC.disable
       open(@pid_file, "wb"){|fh| fh.write Process.pid.to_s } # pidをファイルに保存
+      @is_pid_file_create = true # pidファイルを作ったことを記録
       
 #      @gserver.start(@server_max_connections)
       @gserver.start(@server_max_connections, 20) # backlogを20に設定
@@ -286,9 +291,9 @@ class DangoServerFramework
         sock.binmode
         sock.sync = true
         
-        session[:peer_addr] = sock.peeraddr
+        session[:peer_addr]       = sock.peeraddr
         session[:peer_ip_address] = sock.peeraddr[3]
-        session[:peer_host] = sock.peeraddr[2]
+        session[:peer_host]       = sock.peeraddr[2]
       end
       
       session[:encode_type] = DefaultEncodeType
@@ -306,6 +311,8 @@ class DangoServerFramework
         @arr_send_response.delete_if{|r| r[:time] < Time.now - 60}
       end
       
+      logger.debug "thread_main:do main loop."
+      
       # メインループ入り
       loop do
         if sock.closed?
@@ -322,9 +329,8 @@ class DangoServerFramework
         end
         
         if is_flash_policy_file # Flashのポリシーファイルが来たら
-          @mutex_sock[sock.object_id].synchronize do
-            do_flash_policy_file(sid, sock)
-          end
+          logger.debug "is_flash_policy_file"
+          do_flash_policy_file(sid, sock)
           
           # この部分はポリシーファイルを返す設定に関係なく実行する部分
           sleep 0.1
@@ -421,36 +427,40 @@ class DangoServerFramework
       logger.error "Exception #{sid} #{error_message($!, 'u')}"
       
     ensure
+      # 接続切断の処理
       
-      @mutex_proc_thread.synchronize do
-        begin
-          if !is_flash_policy_file # Flash Policy Fileなら切断処理不要
-            logger.info "#{sock.inspect} #{sid} is closing"
-            if session[:connected] # dango_connect()が実行済みなら
-              dango_close() # 接続解除時に呼び出されるメソッド
-            end
-          end
-        rescue Exception
-          logger.error "#{sock.inspect} #{sid} ERROR\n#{error_message($!, 'u')}"
-        ensure
-          socket_list.delete(sid)     # ソケットリストから削除
+      if !is_flash_policy_file # Flash Policy Fileなら切断処理不要
+        logger.info "#{sock.inspect} #{sid} is closing"
+        if session[:connected] # dango_connect()が実行済みなら
           begin
-            @mutex_sock[sock.object_id].synchronize do
-              sock.close            # ソケットを閉じる
-            end
-          rescue IOError
-            logger.info "IOError #{sock.inspect} #{sid}"
+            @mutex_proc_thread.synchronize do
+              dango_close() # 接続解除時に呼び出されるメソッド
+            end # mutex_proc_thread
           rescue Exception
-            logger.error "#{sock.inspect} #{sid} Exception\n#{error_message($!, 'u')}"
-          end
-          
-          logger.info "#{sock.inspect} #{sid} is closed"
-          @mutex_sock[sock.object_id].synchronize do
-            sock = nil
+            logger.error "#{sock.inspect} #{sid} ERROR\n#{error_message($!, 'u')}"
           end
-          @mutex_sock.delete(sock.object_id)
         end
-      end # mutex_proc_thread
+      end
+      
+      begin
+        @mutex_sock[sock.object_id].synchronize do
+          sock.close            # ソケットを閉じる
+        end
+      rescue IOError
+        logger.info "IOError #{sock.inspect} #{sid}"
+      rescue Exception
+        logger.error "#{sock.inspect} #{sid} Exception\n#{error_message($!, 'u')}"
+      end
+      
+      logger.info "#{sock.inspect} #{sid} is closed"
+      
+      socket_list.delete(sid)     # ソケットリストから削除
+      
+      @mutex_sock[sock.object_id].synchronize do
+        sock = nil   # ソケット削除
+      end
+      
+      @mutex_sock.delete(sock.object_id) # ソケット排他処理から削除
       
       @session_manager.close_session(sid) # sessionの終了処理
     end
@@ -459,11 +469,10 @@ class DangoServerFramework
   
   # Flashのポリシーファイルの処理
   def do_flash_policy_file(sid, sock) 
-    logger.debug "is_flash_policy_file"
     
     if @policy_file_request # ポリシーファイルを返す設定なら
       logger.info "is_flash_policy_file #{sid} #{@policy_file_request}"
-
+      
       allow_str = ""
       
       @policy_file_allow_domain.each do |one_host|
@@ -479,8 +488,11 @@ class DangoServerFramework
 EOF
 
       logger.debug policy_data
-      sock.write policy_data
-      sock.flush
+      @mutex_sock[sock.object_id].synchronize do
+        sock.write policy_data
+        sock.flush
+      end
+      
       logger.info "policy file sent."
     end
   end
@@ -636,14 +648,14 @@ EOF
         begin
           sleep @gc_interval_sec
           
-#          Thread.critical = true
-          logger.debug "GC start "
-          gc_start_time = Time.now
-          GC.enable
-          GC.start 
-          GC.disable
-          logger.debug "GC end #{Time.now - gc_start_time}sec "
-#          Thread.critical = false
+          @mutex_proc_thread.synchronize do
+            logger.debug "GC start "
+            gc_start_time = Time.now
+            GC.enable
+            GC.start 
+            GC.disable
+            logger.debug "GC end #{Time.now - gc_start_time}sec "
+          end
           
           output_memory_statistics() if @statistics_process_memory # メモリ使用量統計
           

data/lib/dango/shared/memory_store.rb

@@ -11,8 +11,8 @@ module DangoFrameworkModule
     def initialize(config)
       @config = config
       @data = {}
-#      @mutex = {}
-      @mutex = Mutex.new
+      @mutex = {}
+#      @mutex = Mutex.new
       
       @transact_key = {}
       @transact_thread = {}
@@ -66,8 +66,8 @@ module DangoFrameworkModule
         raise(DangoFrameworkException, "Shared key is not exist.")
       end
       
-#      @mutex[key].synchronize do
-      @mutex.synchronize do
+      @mutex[key].synchronize do
+#      @mutex.synchronize do
         @transact_key[Thread.current] = key # 違うスレッドからcommitできないように
         
         raise("nested transaction #{key}") if @transact_thread.has_key?(key)
@@ -148,8 +148,8 @@ module DangoFrameworkModule
 #        raise(DangoFrameworkException, "Shared key is not exist.")
       end
       
-#      @mutex[key].synchronize do
-      @mutex.synchronize do
+      @mutex[key].synchronize do
+#      @mutex.synchronize do
         @data[key].deep_dup
       end
     end
@@ -164,13 +164,13 @@ module DangoFrameworkModule
       key = key.to_s
       
       # 未定義なら定義する
-#      if !@data.has_key?(key)
-#        @mutex[key] = Mutex.new
-#      end
+      if !@data.has_key?(key)
+        @mutex[key] = Mutex.new
+      end
       
       # ロックしてデータを入れる
-#      @mutex[key].synchronize do
-      @mutex.synchronize do
+      @mutex[key].synchronize do
+#      @mutex.synchronize do
         @data[key] = value.deep_dup
       end
       

data/lib/dango/version.rb

@@ -2,7 +2,7 @@ module Dango #:nodoc:
   module VERSION #:nodoc:
     MAJOR = 0
     MINOR = 4
-    TINY  = 5
+    TINY  = 6
 
     STRING = [MAJOR, MINOR, TINY].join('.')
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: dango
 version: !ruby/object:Gem::Version 
-  version: 0.4.5
+  version: 0.4.6
 platform: ruby
 authors: 
 - Keisuke Minami
@@ -30,7 +30,7 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        version: 0.4.5
+        version: 0.4.6
     version: 
 - !ruby/object:Gem::Dependency 
   name: hoe