Skip to content
Snippets Groups Projects
Unverified Commit f402ad59 authored by Samuel Couillard's avatar Samuel Couillard Committed by GitHub
Browse files

fix(keycloak): user redirection on KC login (#5184)

* fix(keycloak): user redirection on KC login

* fix specs
parent 9b48e322
No related branches found
No related tags found
No related merge requests found
......@@ -69,7 +69,7 @@ class ExternalController < ApplicationController
cookies.delete(:location)
return redirect_to redirect_location if redirect_location&.match?('\A\/rooms\/\w{3}-\w{3}-\w{3}-\w{3}\/join\z')
redirect_to '/rooms'
redirect_to '/'
rescue StandardError => e
Rails.logger.error("Error during authentication: #{e}")
redirect_to '/?error=SignupError'
......
......@@ -54,7 +54,7 @@ RSpec.describe ExternalController, type: :controller do
get :create_user, params: { provider: 'openid_connect' }
expect(session[:session_token]).to eq(User.find_by(email: OmniAuth.config.mock_auth[:openid_connect][:info][:email]).session_token)
expect(response).to redirect_to('/rooms')
expect(response).to redirect_to('/')
end
it 'assigns the User role to the user' do
......@@ -95,7 +95,7 @@ RSpec.describe ExternalController, type: :controller do
}
get :create_user, params: { provider: 'openid_connect' }
expect(response).to redirect_to('/rooms')
expect(response).to redirect_to('/')
end
it 'doesnt redirect if it doesnt match a room joins format check 2' do
......@@ -107,7 +107,7 @@ RSpec.describe ExternalController, type: :controller do
}
get :create_user, params: { provider: 'openid_connect' }
expect(response).to redirect_to('/rooms')
expect(response).to redirect_to('/')
end
it 'doesnt redirect if it doesnt match a room joins format check 3' do
......@@ -119,7 +119,7 @@ RSpec.describe ExternalController, type: :controller do
}
get :create_user, params: { provider: 'openid_connect' }
expect(response).to redirect_to('/rooms')
expect(response).to redirect_to('/')
end
it 'deletes the cookie after reading' do
......@@ -215,7 +215,7 @@ RSpec.describe ExternalController, type: :controller do
create(:user, external_id: OmniAuth.config.mock_auth[:openid_connect][:uid])
expect { get :create_user, params: { provider: 'openid_connect' } }.not_to raise_error
expect(response).to redirect_to('/rooms')
expect(response).to redirect_to('/')
end
it 'returns an InviteInvalid error if no invite is passed' do
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment