Skip to content

Commit 9eef3d7

Browse files
author
kaptron
committed
Adds support for confirmationRedirectUri param to pass to IdeoSSO
1 parent 90ba517 commit 9eef3d7

File tree

7 files changed

+26
-7
lines changed

7 files changed

+26
-7
lines changed

__mocks__/ideo-sso.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
export default {
22
baseApiUrl: '',
33
profileUrl: '',
4+
init: jest.fn(),
5+
initFromEnv: jest.fn(),
46
}

app/controllers/home_controller.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ def capture_email_and_token
6161
def store_redirect_param
6262
return if params[:redirect].blank?
6363

64+
@redirect = params[:redirect]
6465
redirect_uri = clean_redirect
6566
store_location_for :user, redirect_uri
6667
load_redirect_organization_from_url(redirect_uri)

app/javascript/packs/ideo-sso.js

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,28 @@ function init() {
55
if (!window.IDEO_SSO_PARAMS) {
66
return
77
}
8-
const { action, email, token, logoutRequired } = window.IDEO_SSO_PARAMS
8+
const {
9+
action,
10+
email,
11+
token,
12+
logoutRequired,
13+
redirect,
14+
loginUrl,
15+
} = window.IDEO_SSO_PARAMS
16+
17+
let confirmationRedirectUri = loginUrl
18+
if (redirect) {
19+
// append the redirect to the original oauth redirect path
20+
confirmationRedirectUri += `?redirect=${redirect}`
21+
}
922

1023
if (action === 'signIn') {
11-
IdeoSSO.signIn({ email })
24+
IdeoSSO.signIn({ email, confirmationRedirectUri })
1225
return
1326
}
1427
if (action === 'signUp') {
1528
const signUp = () => {
16-
IdeoSSO.signUp({ email, token })
29+
IdeoSSO.signUp({ email, token, confirmationRedirectUri })
1730
}
1831

1932
if (logoutRequired) {

app/views/home/login.html.erb

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22
window.IDEO_SSO_PARAMS = {
33
action: 'signIn',
44
email: '<%= @email %>',
5+
redirect: '<%= @redirect %>',
6+
loginUrl: '<%= login_url %>',
57
}
68
</script>
79

app/views/home/sign_up.html.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
email: '<%= @email %>',
55
token: '<%= @token %>',
66
logoutRequired: <%= @user_was_signed_in %>,
7+
loginUrl: '<%= login_url %>',
78
}
89
</script>
910

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@
122122
"hypher": "^0.2.5",
123123
"i18n-iso-countries": "3.7.3",
124124
"identity-obj-proxy": "^3.0.0",
125-
"ideo-sso-js-sdk": "ideo/sso-js-sdk#v2.1.0",
125+
"ideo-sso-js-sdk": "ideo/sso-js-sdk#v2.1.1",
126126
"if-emoji": "^0.1.0",
127127
"jest-styled-components": "7.0.0-2",
128128
"js-cookie": "^2.2.1",

yarn.lock

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7578,9 +7578,9 @@ identity-obj-proxy@^3.0.0:
75787578
dependencies:
75797579
harmony-reflect "^1.4.6"
75807580

7581-
ideo-sso-js-sdk@ideo/sso-js-sdk#v2.1.0:
7582-
version "2.1.0"
7583-
resolved "https://codeload.github.com/ideo/sso-js-sdk/tar.gz/fd66521d0ebb391f69c0dd5e2519e0e57b20d106"
7581+
ideo-sso-js-sdk@ideo/sso-js-sdk#v2.1.1:
7582+
version "2.1.1"
7583+
resolved "https://codeload.github.com/ideo/sso-js-sdk/tar.gz/d73606ae8b59f8e99fb457f5e3c42e7a4dfd303d"
75847584
dependencies:
75857585
babel-polyfill "^6.26.0"
75867586
js-cookie "^2.2.0"

0 commit comments

Comments
 (0)