Commit aa6f3db2 authored by Nikolay Belokolodov's avatar Nikolay Belokolodov Committed by Sean McGivern

Fix SAML SSO redirects for pseudonymized URLS

The Group SSO Controller is skipping the :group method before it executes.
The URL-pseudonymization code that gets included on every page is checking if group is defined.
By calling group for tracking the pseudonymized URL,
it caused an endless redirect.

Changelog: fixed
parent d0dbbf23
...@@ -75,6 +75,10 @@ class Groups::SsoController < Groups::ApplicationController ...@@ -75,6 +75,10 @@ class Groups::SsoController < Groups::ApplicationController
private private
def group
@group ||= unauthenticated_group
end
def new_user def new_user
@new_user ||= User.new(new_user_params) @new_user ||= User.new(new_user_params)
end end
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
require 'spec_helper' require 'spec_helper'
RSpec.describe 'Group SAML SSO', :js do RSpec.describe 'Group SAML SSO', :js, :snowplow do
include CookieHelper include CookieHelper
let(:user) { create(:user) } let(:user) { create(:user) }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment