github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-01-18 16:52:45 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

the protection I placed was in the wrong path moved to /session/sso

Browse Source 
correct previous commit
This commit is contained in:
Sam 2018-11-09 17:17:43 +11:00
parent 5076487eaf
commit 64d9be726f
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/controllers/session_controller.rb
View File

@ -173,7 +173,7 @@ class SessionController < ApplicationController
end
# never redirects back to sso in an sso loop
if return_path.start_with?(path("/sso"))
if return_path.start_with?(path("/session/sso"))
return_path = path("/")
end

6
spec/requests/session_controller_spec.rb
View File

@ -303,8 +303,8 @@ RSpec.describe SessionController do
end
it 'will never redirect back to /sso path' do
sso = get_sso("/sso?bla=1")
it 'will never redirect back to /session/sso path' do
sso = get_sso("/session/sso?bla=1")
sso.email = user.email
sso.external_id = 'abc'
sso.username = 'sam'
@ -312,7 +312,7 @@ RSpec.describe SessionController do
get "/session/sso_login", params: Rack::Utils.parse_query(sso.payload), headers: headers
expect(response).to redirect_to('/')
sso = get_sso("http://#{Discourse.current_hostname}/sso?bla=1")
sso = get_sso("http://#{Discourse.current_hostname}/session/sso?bla=1")
sso.email = user.email
sso.external_id = 'abc'
sso.username = 'sam'