[bbb-greenlight: 6/12] Fixed issue with ldap redirect throwing 404 (#2231)




commit 49f7a7714abd1af86a1da292f4e67bd3afbc1e12
Author: Ahmad Farhat <ahmad af farhat gmail com>
Date:   Mon Oct 26 11:50:43 2020 -0400

    Fixed issue with ldap redirect throwing 404 (#2231)

 app/controllers/concerns/authenticator.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
---
diff --git a/app/controllers/concerns/authenticator.rb b/app/controllers/concerns/authenticator.rb
index b2802da..bf2534f 100644
--- a/app/controllers/concerns/authenticator.rb
+++ b/app/controllers/concerns/authenticator.rb
@@ -42,7 +42,7 @@ module Authenticator
       redirect_to admins_path
     elsif user.activated?
       # Dont redirect to any of these urls
-      dont_redirect_to = [root_url, signin_url, ldap_signin_url, signup_url, unauthorized_url,
+      dont_redirect_to = [root_url, signin_url, ldap_signin_url, ldap_callback_url, signup_url, 
unauthorized_url,
                           internal_error_url, not_found_url]
       url = if cookies[:return_to] && !dont_redirect_to.include?(cookies[:return_to])
         cookies[:return_to]
@@ -62,7 +62,7 @@ module Authenticator
   end
 
   def ensure_unauthenticated_except_twitter
-    redirect_to current_user.main_room if current_user && params[:old_twitter_user_id].nil?
+    redirect_to current_user.main_room || root_path if current_user && params[:old_twitter_user_id].nil?
   end
 
   # Logs current user out of GreenLight.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]