omniauth-applicaster 1.3.3 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 28a4d1d112c3be8042347abdeb5854641c7c0bbd
-  data.tar.gz: 5862c3cf30313cbdf6f8866db6be8f8e34acbe93
+  metadata.gz: 4e9074ea4ccb8f9890af08d92a9d545fc4b1aae1
+  data.tar.gz: d1815e9a3809d90d62c60f4987213a36c71f89eb
 SHA512:
-  metadata.gz: b5d12249370ea652c63238a3524f3287f751b25744a3137063d294c86408493139dfcafd81ba41d1f95ea59d61a4e3e796437550db7ba8c31831608c69d5e61d
-  data.tar.gz: 31d8eb9dec37665b13864f22fbbc90093c8db27acf068b4a13eac67682b50025a5b4cdf9ade6e00673b336fb5af5a2fc79fba7cad5f3f84ded1670638ceaf99e
+  metadata.gz: 8029917d6ee422a7b8d7870345afefaba077276014774dea91b25891720ca4a4d9714824032ff7dc2e4246e2505a4afd7203589057acef372e9c635033867c23
+  data.tar.gz: 3b58c45ac99eb64f24d7290ae11eb90d604631ee95276c5bf810a0cb7c2f04d38e7e0a5366c4b9585f67ab73802a008de6d54be1a6834fdbe0694680e09cde62

data/lib/applicaster/auth_helpers.rb

@@ -10,7 +10,7 @@ module Applicaster
 
     def authenticate_user!
       unless current_user
-        session[:path_before_login] = url_for(params)
+        store_location!
         redirect_to '/auth/applicaster'
       end
     end
@@ -23,6 +23,14 @@ module Applicaster
 
     protected
 
+    def store_location!
+      session[:path_before_login] = if request.get?
+                                      request.fullpath
+                                    else
+                                      request.referrer
+                                    end
+    end
+
     def clear_omniauth_credentials
       session.delete(:omniauth_credentials)
     end

data/lib/omniauth-applicaster/version.rb

@@ -1,5 +1,5 @@
 module OmniAuth
   module Applicaster
-    VERSION = "1.3.3"
+    VERSION = "1.3.4"
   end
 end

data/spec/lib/applicaster/auth_helpers_spec.rb

@@ -1,9 +1,11 @@
 RSpec.describe Applicaster::AuthHelpers do
   let(:dummy_class) { Class.new(DummyController) { include Applicaster::AuthHelpers } }
   let(:controller) { dummy_class.new }
+  let(:request) { double("request", get?: true) }
 
   before do
     allow(controller).to receive(:session).and_return(session)
+    allow(controller).to receive(:request) { request }
 
     stub_current_user_requests
   end
@@ -76,7 +78,7 @@ RSpec.describe Applicaster::AuthHelpers do
     context "when current_user is nil" do
       before do
         allow(controller).to receive(:current_user).and_return(nil)
-        allow(controller).to receive(:url_for).and_return("/current")
+        allow(request).to receive(:fullpath).and_return("/current")
       end
 
       it "redirects to '/auth/applicaster'" do
@@ -89,6 +91,18 @@ RSpec.describe Applicaster::AuthHelpers do
 
         expect(controller.session[:path_before_login]).to eq("/current")
       end
+
+      context "when request is not GET" do
+        before do
+          allow(request).to receive(:get?) { false }
+          allow(request).to receive(:referrer) { "/referrer" }
+        end
+
+        it "redirects to the referrer of the request" do
+          controller.authenticate_user!
+          expect(controller.session[:path_before_login]).to eq("/referrer")
+        end
+      end
     end
   end
 

data/spec/support/dummy_controller.rb

@@ -12,4 +12,7 @@ class DummyController
   def params
     {}
   end
+
+  def request
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-applicaster
 version: !ruby/object:Gem::Version
-  version: 1.3.3
+  version: 1.3.4
 platform: ruby
 authors:
 - Neer Friedman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-28 00:00:00.000000000 Z
+date: 2015-02-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler