Merge pull request #8 from jcmuller/add_option_to_redirect

Add ability to override where to redirect to after login/out

Author: rossta

.travis.yml

@@ -9,6 +9,8 @@ env:
   - secure: oQs2AG/S1KoECnRqrg8htnB+D8lbrphR2RCz8GbYkCzK8ksR0scXM/kBnhV0oTel1H2mMPU29Gopr/uL+gcExVTF+nMOGZh5guTpZdWD49AnWQqIdKGlpyzOBGHPw3A1a9h50lLQi/CEmoDkeAMwA0pAFt5MRgLELPU9O3p/t8Q=
   - secure: TUP2EXBw9M+pRSfjfFNKpcr8cxksg38tco4NvLsHZE5skuzMsxTaL/KG0gf7/GqQI1xxpFwj2SFD3NtKRDf5mJHeRQtsVPHznnFJ1GtekVy9PE3/ixppfF5Y32JIYFbjBxoZ9Qfa9P7aY8tBnfShzXw6yGkzsEApIT35WnGCbbM=
 rvm:
+- 2.1.0
 - 2.0.0
 - 1.9.3
 before_script: sh -c 'cd spec/dummy && bundle exec rake db:create db:migrate'
+cache: bundler

Gemfile

@@ -21,4 +21,5 @@ else
 end
 
 gem "pry"
-gem "pry-debugger"
+gem "pry-debugger", :platforms => :mri_19
+gem "pry-byebug", :platforms => [:mri_20, :mri_21]

README.md

@@ -34,6 +34,10 @@ OpenSesame.configure do |config|
   config.github ENV['GITHUB_APP_ID'], ENV['GITHUB_SECRET']
   config.organization 'challengepost'
   config.mounted_at   '/opensesame'
+
+  config.redirect_to '/path' # Set redirect to for both login and logout
+  config.redirect_after_login '/path'
+  config.redirect_after_logout '/path'
 end
 ```
 

app/controllers/open_sesame/sessions_controller.rb

@@ -15,12 +15,12 @@ def new
 
     def create
       login_opensesame
-      redirect_to main_app.root_url
+      redirect_to after_login_path
     end
 
     def destroy
       logout_opensesame
-      redirect_to main_app.root_url
+      redirect_to after_logout_path
     end
 
     def failure
@@ -68,5 +68,13 @@ def logout_opensesame
       flash[:notice] = "Logged out!"
     end
 
+    def after_login_path
+      OpenSesame.after_login_path || main_app.root_url
+    end
+
+    def after_logout_path
+      OpenSesame.after_logout_path || main_app.root_url
+    end
+
   end
 end

lib/open_sesame/configuration.rb

@@ -10,7 +10,9 @@ class Configuration
       :enabled,
       :debug,
       :full_host,
-      :auto_access_provider
+      :auto_access_provider,
+      :after_login_path,
+      :after_logout_path
     ]
     attr_accessor *CONFIGURABLE_ATTRIBUTES
 
@@ -50,6 +52,19 @@ def enable(enabled)
       self.enabled = !!enabled
     end
 
+    def redirect_to(to)
+      self.after_login_path = to
+      self.after_logout_path = to
+    end
+
+    def redirect_after_login(to)
+      self.after_login_path = to
+    end
+
+    def redirect_after_logout(to)
+      self.after_logout_path = to
+    end
+
     def enabled?
       !!self.enabled
     end

lib/open_sesame/engine.rb

@@ -16,7 +16,7 @@ class Engine < ::Rails::Engine
       include OpenSesame::Helpers::ViewHelper
     end
 
-    initializer "openseseame precompile" do |app|
+    initializer "opensesame.precompile" do |app|
       app.config.assets.precompile += ['open_sesame/opensesame.css']
     end
 

spec/lib/open_sesame/configuration_spec.rb

@@ -31,6 +31,22 @@
     configuration.auto_access_provider.should == "github"
   end
 
+  it "sets all redirects to" do
+    configuration.redirect_to "/foobar"
+    configuration.after_login_path.should == "/foobar"
+    configuration.after_logout_path.should == "/foobar"
+  end
+
+  it "sets after_login path" do
+    configuration.redirect_after_login "/foobar"
+    configuration.after_login_path.should == "/foobar"
+  end
+
+  it "sets after_logout redirect_to" do
+    configuration.redirect_after_logout "/foobar"
+    configuration.after_logout_path.should == "/foobar"
+  end
+
   describe "valid?" do
     it "false when values not set" do
       configuration.organization "challengepost"