Skip to content

Commit

Permalink
5. clean up commented lines
Browse files Browse the repository at this point in the history
  • Loading branch information
yzhoubk committed Jan 30, 2025
1 parent 9a01aff commit 8162aba
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 9 deletions.
3 changes: 0 additions & 3 deletions app/controllers/application_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,6 @@ def allow_geoblacklight_params

# root will be used as new_session_path
# Shim because we're not using Devise's :database_authenticatable
# def new_session_path(_scope)
# new_user_session_path
# end

# @param [String] return_url address that calnet will redirect to post-logout
# @return [String] uri to the calnet sso page
Expand Down
4 changes: 0 additions & 4 deletions app/controllers/sessions_controller.rb
Original file line number Diff line number Diff line change
@@ -1,8 +1,4 @@
class SessionsController < ApplicationController
# def new
# return_url = params[:url] || request.referer || root_path
# redirect_to user_calnet_omniauth_authorize_path(url: return_url)
# end

def destroy
sign_out current_user if signed_in?
Expand Down
1 change: 0 additions & 1 deletion config/initializers/devise.rb
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,6 @@
config.sign_out_via = :get
config.omniauth :cas,
name: :calnet,
# url: "https://auth#{'-test' unless Rails.env.production?}.berkeley.edu",
host: "auth#{'-test' unless Rails.env.production?}.berkeley.edu",
login_url: '/cas/login',
logout_url: '/cas/logout',
Expand Down
1 change: 0 additions & 1 deletion config/routes.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@

devise_for :users, controllers: { omniauth_callbacks: 'omniauth_callbacks' }
devise_scope :user do
# get 'users/sign_in', to: 'sessions#new', as: :new_user_session
get 'users/sign_out', to: 'sessions#destroy', as: :destroy_user_session
end

Expand Down

0 comments on commit 8162aba

Please sign in to comment.