galapagos_mobile 0.0.7 → 0.0.8

data/README.md

@@ -6,9 +6,9 @@ jpmobileプラグインを使用した場合に発生する不具合を解消す
 
 以下の環境で動作確認
 
-* Rails : 3.0.4
-* jpmobile : 0.1.4
-* devise : 1.1.5 〜 1.1.6 （利用する場合）
+* Rails : 3.0.4 〜 3.0.6
+* jpmobile : 0.1.4 〜 1.0.0.pre
+* devise : 1.1.5 〜 1.1.8 （利用する場合）
 
 ## install
 
@@ -36,10 +36,31 @@ setupメソッドを実行
     $ rails plugin install git://github.com/matsuda/galapagos_mobile.git
 
 
+## Deviseを使用する場合
+
+deviseの初期設定した際に生成される設定ファイル(devise.rb)に以下を設定
+
+    # /config/initializers/devise.rb 
+    Devise.setup do |config| 
+      ... 
+      config.warden do |manager| 
+        manager.failure_app = GalapagosMobile::FailureApp
+      end 
+    end
+
+FailureAppをカスマイズする場合は、
+
+    class CustomFailureApp < GalapagosMobile::FailureApp
+    end
+
+を継承して上記のwarden managerにセットする。
+
 ## 実装されている機能
 
 * Cookie非対応の携帯でログイン処理を可能にする。
 
-* Deviseプラグインを利用している場合に authenticate_{scope}! メソッドで　invalidだった場合にsession_idを引き継がせる。  
+### deviseを使用している場合
 
-login画面へのリダイレクト時にsession_idを付与し、login後に元の画面へリダイレクトバックする時に古いsession_idを削除する。
+* login画面へのリダイレクト時にsession\_idを付与
+* login後に元の画面へリダイレクトバックする時に古いsession\_idを削除する。
+* authenticate_{scope}! メソッドで　invalidだった場合にsession\_idを引き継がせる。  

data/lib/galapagos_mobile/devise.rb

@@ -2,44 +2,6 @@
 # monkey patching for jpmobile 0.1.4 + devise 1.1.6
 module GalapagosMobile
   module Devise
-    module FailureApp
-      extend ActiveSupport::Concern
-
-      included do
-        class_inheritable_accessor :trans_sid_mode
-        # Set trans_sid_mode if you want to use others.
-        # ex. ::Devise::FailureApp.trans_sid_mode = :always
-        #     in config/initializers/galapagos_mobile.rb
-        self.trans_sid_mode = :mobile
-      end
-
-      module InstanceMethods
-        # from jpmobile/trans_sid.rb -v 0.1.4
-        protected
-        # URLにsession_idを追加する。
-        def default_url_options
-          result = super || {}
-          return result unless request # for test process
-          return result unless apply_trans_sid?
-          return result.merge({session_key => jpmobile_session_id})
-        end
-
-        private
-        # session_keyを返す。
-        def session_key
-          unless key = Rails.application.config.session_options.merge(request.session_options || {})[:key]
-            key = ActionDispatch::Session::AbstractStore::DEFAULT_OPTIONS[:key]
-          end
-          key
-        end
-
-        # session_idを返す
-        def jpmobile_session_id
-          request.session_options[:id] rescue session.session_id
-        end
-      end
-    end
-
     module Controllers
       module Helpers
         extend ActiveSupport::Concern
@@ -70,7 +32,6 @@ module ActionDispatch::Routing
     # need devise_for mappings already declared to create filters and helpers.
     def finalize_with_galapagos_mobile!
       finalize_without_galapagos_mobile!
-      ::Devise::FailureApp.send :include, GalapagosMobile::Devise::FailureApp
       ActionController::Base.send :include, GalapagosMobile::Devise::Controllers::Helpers
     end
     alias_method_chain :finalize!, :galapagos_mobile

data/lib/galapagos_mobile/failure_app.rb

@@ -0,0 +1,64 @@
+# -*- coding: utf-8 -*-
+# 
+# /config/initializers/devise.rb 
+# Devise.setup do |config| 
+#   ... 
+#   config.warden do |manager| 
+#     manager.failure_app = GalapagosMobile::FailureApp
+#   end 
+# end
+# 
+# resources
+#   from jpmobile/trans_sid.rb -v 1.0.0.pre
+#   from devise/failure_app.rb -v 0.1.8
+# 
+module GalapagosMobile
+  class FailureApp < ::Devise::FailureApp
+    class_inheritable_accessor :trans_sid_mode
+    # Set trans_sid_mode if you want to use others.
+    # ex. ::Devise::FailureApp.trans_sid_mode = :always
+    #     in config/initializers/galapagos_mobile.rb
+    self.trans_sid_mode = :mobile
+
+    protected
+    # URLにsession_idを追加する。
+    def default_url_options
+      result = super || {}
+      return result unless request # for test process
+      return result unless apply_trans_sid?
+      return result.merge({session_key => jpmobile_session_id})
+    end
+
+    # override
+    def http_auth?
+      if request.xhr?
+        ::Devise.http_authenticatable_on_xhr
+      elsif http_auth_header?
+        !(request_format && ::Devise.navigational_formats.include?(request_format))
+      else
+        false
+      end
+    end
+
+    # from Devise -v 1.2.0
+    # It does not make sense to send authenticate headers in ajax requests
+    # or if the user disabled them.
+    def http_auth_header?
+      ::Devise.mappings[scope].to.http_authenticatable && !request.xhr?
+    end
+
+    private
+    # session_keyを返す。
+    def session_key
+      unless key = Rails.application.config.session_options.merge(request.session_options || {})[:key]
+        key = ActionDispatch::Session::AbstractStore::DEFAULT_OPTIONS[:key]
+      end
+      key
+    end
+
+    # session_idを返す
+    def jpmobile_session_id
+      request.session_options[:id] rescue session.session_id
+    end
+  end
+end

data/lib/galapagos_mobile/jpmobile.rb

@@ -1,3 +1,4 @@
+# -*- coding: utf-8 -*-
 begin
   require 'jpmobile'
 rescue LoadError, NameError => e
@@ -7,9 +8,11 @@ rescue LoadError, NameError => e
 end
 
 ActiveSupport.on_load(:action_controller) do
+  require 'galapagos_mobile/request'
   require 'galapagos_mobile/trans_sid'
 end
 
 if defined?(Devise)
   require 'galapagos_mobile/devise'
+  require 'galapagos_mobile/failure_app'
 end

data/lib/galapagos_mobile/request.rb

@@ -0,0 +1,15 @@
+# -*- coding: utf-8 -*-
+class ActionDispatch::Request
+  def reset_session_with_jpmobile
+    reset_session_without_jpmobile
+    # SSLと非SSLでとsession_idが異なる場合に、
+    # Cookie非対応の携帯でもsession_idをパラメータで引き回すための対応
+    if self.mobile? # && !self.mobile.supports_cookie?
+      self.session_options[:id] = ActiveSupport::SecureRandom.hex(16)
+      if Rails.application.config.session_store == ActiveRecord::SessionStore
+        @env[ActiveRecord::SessionStore::SESSION_RECORD_KEY] = nil
+      end
+    end
+  end
+  alias_method_chain :reset_session, :jpmobile
+end

data/lib/galapagos_mobile/trans_sid.rb

@@ -1,15 +1,28 @@
 # -*- coding: utf-8 -*-
-class ActionDispatch::Request
-  def reset_session_with_jpmobile
-    reset_session_without_jpmobile
-    # DocomoでSSLを使用するとsession_idが変更されるため、
-    # Cookie非対応の携帯でもsession_idをパラメータで引き回す。
-    if self.mobile? # && !self.mobile.supports_cookie?
-      self.session_options[:id] = ActiveSupport::SecureRandom.hex(16)
-      if Rails.application.config.session_store == ActiveRecord::SessionStore
-        @env[ActiveRecord::SessionStore::SESSION_RECORD_KEY] = nil
+# from jpmobile -v 1.0.0.pre
+require 'jpmobile/trans_sid.rb'
+module ActionController
+  class Metal #:nodoc:
+    private
+    # trans_sidを適用すべきかを返す。
+    def apply_trans_sid?
+      # session_id が blank の場合は適用しない
+      return false if trans_sid_mode and jpmobile_session_id.blank?
+
+      case trans_sid_mode
+      when :always
+        session.inspect
+        return true
+      when :mobile
+        # SSLと非SSLでとsession_idが異なる場合に、
+        # Cookie非対応の携帯でもsession_idをパラメータで引き回すための対応
+        if request.mobile?# and !request.mobile.supports_cookie?
+          session.inspect
+          return true
+        end
       end
+
+      return false
     end
   end
-  alias_method_chain :reset_session, :jpmobile
 end

data/lib/galapagos_mobile/version.rb

@@ -1,3 +1,3 @@
 module GalapagosMobile
-  VERSION = "0.0.7"
+  VERSION = "0.0.8"
 end

metadata

@@ -2,7 +2,7 @@
 name: galapagos_mobile
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.0.7
+  version: 0.0.8
 platform: ruby
 authors: 
 - Kosuke Matsuda
@@ -35,8 +35,10 @@ files:
 - install.rb
 - lib/galapagos_mobile.rb
 - lib/galapagos_mobile/devise.rb
+- lib/galapagos_mobile/failure_app.rb
 - lib/galapagos_mobile/jpmobile.rb
 - lib/galapagos_mobile/rails.rb
+- lib/galapagos_mobile/request.rb
 - lib/galapagos_mobile/trans_sid.rb
 - lib/galapagos_mobile/version.rb
 - test/galapagos_mobile_test.rb