Merge pull request #90 from mlibrary/fix-logout

changes logout url to redirect to oauth2/sign_out

Author: niquerio

get-this.rb

@@ -34,7 +34,9 @@ def set_patron
 
 get "/logout" do
   session.clear
-  redirect "https://shibboleth.umich.edu/cgi-bin/logout?https://lib.umich.edu/"
+  weblogin_logout_url = URI.encode_www_form_component("https://shibboleth.umich.edu/cgi-bin/logout?https://lib.umich.edu/")
+  log_out_url = "/oauth2/sign_out?rd=#{weblogin_logout_url}"
+  redirect log_out_url
 end
 
 before do

spec/weblogin_spec.rb

@@ -44,10 +44,10 @@
   end
 
   context "/logout" do
-    it "clears the session and redirects to shibboleth" do
+    it "clears the session and redirects to oauth2/sign_out" do
       get "/logout"
       expect(last_request.env["rack.session"][:uniqname]).to be_nil
-      expect(last_response.location).to eq("https://shibboleth.umich.edu/cgi-bin/logout?https://lib.umich.edu/")
+      expect(last_response.location).to include("oauth2/sign_out")
     end
   end
 end