UI: Test business logic for oidc callback params (#19727)
Co-authored-by: claire bontempo <68122737+hellobontempo@users.noreply.github.com>
This commit is contained in:
Chelsea Shaw
GitHub
parent
fe53ebd19c
commit
ea99048639
|
|
@ -5,33 +5,39 @@
|
|||
|
||||
import Route from '@ember/routing/route';
|
||||
|
||||
export default Route.extend({
|
||||
templateName: 'vault/cluster/oidc-callback',
|
||||
model() {
|
||||
// left blank so we render the template immediately
|
||||
},
|
||||
afterModel() {
|
||||
let { auth_path: path, code, state } = this.paramsFor(this.routeName);
|
||||
let { namespaceQueryParam: namespace } = this.paramsFor('vault.cluster');
|
||||
export function getParamsForCallback(qp, searchString) {
|
||||
const queryString = decodeURIComponent(searchString);
|
||||
let { path, code, state, namespace } = qp;
|
||||
// namespace from state takes precedence over the cluster's ns
|
||||
if (state?.includes(',ns=')) {
|
||||
[state, namespace] = state.split(',ns=');
|
||||
}
|
||||
// some SSO providers do not return a url-encoded state param
|
||||
// check for namespace using URLSearchParams instead of paramsFor
|
||||
const queryString = decodeURIComponent(window.location.search);
|
||||
const urlParams = new URLSearchParams(queryString);
|
||||
const checkState = urlParams.get('state');
|
||||
if (checkState?.includes(',ns=')) {
|
||||
[state, namespace] = checkState.split(',ns=');
|
||||
}
|
||||
path = window.decodeURIComponent(path);
|
||||
const source = 'oidc-callback'; // required by event listener in auth-jwt component
|
||||
const queryParams = { source, path: path || '', code: code || '', state: state || '' };
|
||||
const payload = { source: 'oidc-callback', path: path || '', code: code || '', state: state || '' };
|
||||
if (namespace) {
|
||||
queryParams.namespace = namespace;
|
||||
payload.namespace = namespace;
|
||||
}
|
||||
window.opener.postMessage(queryParams, window.origin);
|
||||
return payload;
|
||||
}
|
||||
|
||||
export default Route.extend({
|
||||
templateName: 'vault/cluster/oidc-callback',
|
||||
model() {
|
||||
// left blank so we render the template immediately
|
||||
},
|
||||
afterModel() {
|
||||
const { auth_path: path, code, state } = this.paramsFor(this.routeName);
|
||||
const { namespaceQueryParam: namespace } = this.paramsFor('vault.cluster');
|
||||
const queryString = window.location.search;
|
||||
const payload = getParamsForCallback({ path, code, state, namespace }, queryString);
|
||||
window.opener.postMessage(payload, window.origin);
|
||||
},
|
||||
setupController(controller) {
|
||||
this._super(...arguments);
|
||||
|
|
|
|||
|
|
@ -3,9 +3,10 @@
|
|||
* SPDX-License-Identifier: MPL-2.0
|
||||
*/
|
||||
|
||||
import { module, skip, test } from 'qunit';
|
||||
import { module, test } from 'qunit';
|
||||
import { setupTest } from 'ember-qunit';
|
||||
import sinon from 'sinon';
|
||||
import { getParamsForCallback } from 'vault/routes/vault/cluster/oidc-callback';
|
||||
|
||||
module('Unit | Route | vault/cluster/oidc-callback', function (hooks) {
|
||||
setupTest(hooks);
|
||||
|
|
@ -52,131 +53,27 @@ module('Unit | Route | vault/cluster/oidc-callback', function (hooks) {
|
|||
assert.ok(this.route);
|
||||
});
|
||||
|
||||
skip('it uses namespace param from state instead of cluster, with custom oidc path', function (assert) {
|
||||
this.routeName = 'vault.cluster.oidc-callback';
|
||||
this.callbackUrlQueryParams(encodeURIComponent(`${this.state},ns=test-ns`));
|
||||
this.route.paramsFor = (path) => {
|
||||
if (path === 'vault.cluster') return { namespaceQueryParam: 'admin' };
|
||||
return {
|
||||
auth_path: 'oidc-dev',
|
||||
code: this.code,
|
||||
module('getParamsForCallback helper fn', function () {
|
||||
test('it parses params correctly with regular inputs and no namespace', function (assert) {
|
||||
const qp = {
|
||||
state: 'my-state',
|
||||
code: 'my-code',
|
||||
path: 'oidc-path',
|
||||
};
|
||||
};
|
||||
this.route.afterModel();
|
||||
assert.propEqual(
|
||||
this.windowStub.lastCall.args[0],
|
||||
{
|
||||
code: this.code,
|
||||
path: 'oidc-dev',
|
||||
namespace: 'test-ns',
|
||||
state: this.state,
|
||||
source: 'oidc-callback',
|
||||
},
|
||||
'ns from state not cluster'
|
||||
);
|
||||
const searchString = `?code=my-code&state=my-state`;
|
||||
const results = getParamsForCallback(qp, searchString);
|
||||
assert.deepEqual(results, { source: 'oidc-callback', ...qp });
|
||||
});
|
||||
|
||||
skip('it uses namespace from cluster when state does not include ns param', function (assert) {
|
||||
this.routeName = 'vault.cluster.oidc-callback';
|
||||
this.callbackUrlQueryParams(encodeURIComponent(this.state));
|
||||
this.route.paramsFor = (path) => {
|
||||
if (path === 'vault.cluster') return { namespaceQueryParam: 'admin' };
|
||||
return {
|
||||
auth_path: this.path,
|
||||
code: this.code,
|
||||
test('it parses params correctly regular inputs and namespace param', function (assert) {
|
||||
const params = {
|
||||
state: 'my-state',
|
||||
code: 'my-code',
|
||||
path: 'oidc-path',
|
||||
namespace: 'my-namespace',
|
||||
};
|
||||
};
|
||||
this.route.afterModel();
|
||||
assert.propEqual(
|
||||
this.windowStub.lastCall.args[0],
|
||||
{
|
||||
code: this.code,
|
||||
path: this.path,
|
||||
namespace: 'admin',
|
||||
state: this.state,
|
||||
source: 'oidc-callback',
|
||||
},
|
||||
`namespace is from cluster's namespaceQueryParam`
|
||||
);
|
||||
});
|
||||
|
||||
skip('it correctly parses encoded, nested ns param from state', function (assert) {
|
||||
this.callbackUrlQueryParams(encodeURIComponent(`${this.state},ns=parent-ns/child-ns`));
|
||||
this.route.afterModel();
|
||||
assert.propEqual(
|
||||
this.windowStub.lastCall.args[0],
|
||||
{
|
||||
code: this.code,
|
||||
path: this.path,
|
||||
namespace: 'parent-ns/child-ns',
|
||||
state: this.state,
|
||||
source: 'oidc-callback',
|
||||
},
|
||||
'it has correct nested ns from state and sets as namespace param'
|
||||
);
|
||||
});
|
||||
|
||||
skip('the afterModel hook returns when both cluster and route params are empty strings', function (assert) {
|
||||
this.routeName = 'vault.cluster.oidc-callback';
|
||||
this.callbackUrlQueryParams('');
|
||||
this.route.paramsFor = (path) => {
|
||||
if (path === 'vault.cluster') return { namespaceQueryParam: '' };
|
||||
return {
|
||||
auth_path: '',
|
||||
code: '',
|
||||
};
|
||||
};
|
||||
this.route.afterModel();
|
||||
assert.propEqual(
|
||||
this.windowStub.lastCall.args[0],
|
||||
{
|
||||
path: '',
|
||||
state: '',
|
||||
code: '',
|
||||
source: 'oidc-callback',
|
||||
},
|
||||
'model hook returns with empty params'
|
||||
);
|
||||
});
|
||||
|
||||
skip('the afterModel hook returns when state param does not exist', function (assert) {
|
||||
this.routeName = 'vault.cluster.oidc-callback';
|
||||
this.callbackUrlQueryParams('stateless');
|
||||
this.route.afterModel();
|
||||
assert.propEqual(
|
||||
this.windowStub.lastCall.args[0],
|
||||
{
|
||||
code: this.code,
|
||||
path: 'oidc',
|
||||
state: '',
|
||||
source: 'oidc-callback',
|
||||
},
|
||||
'model hook returns empty string when state param nonexistent'
|
||||
);
|
||||
});
|
||||
|
||||
skip('the afterModel hook returns when cluster ns exists and all route params are empty strings', function (assert) {
|
||||
this.routeName = 'vault.cluster.oidc-callback';
|
||||
this.callbackUrlQueryParams('');
|
||||
this.route.paramsFor = (path) => {
|
||||
if (path === 'vault.cluster') return { namespaceQueryParam: 'ns1' };
|
||||
return {
|
||||
auth_path: '',
|
||||
code: '',
|
||||
};
|
||||
};
|
||||
this.route.afterModel();
|
||||
assert.propEqual(
|
||||
this.windowStub.lastCall.args[0],
|
||||
{
|
||||
code: '',
|
||||
namespace: 'ns1',
|
||||
path: '',
|
||||
source: 'oidc-callback',
|
||||
state: '',
|
||||
},
|
||||
'model hook returns with empty parameters'
|
||||
);
|
||||
const results = getParamsForCallback(params, '?code=my-code&state=my-state&namespace=my-namespace');
|
||||
assert.deepEqual(results, { source: 'oidc-callback', ...params });
|
||||
});
|
||||
|
||||
/*
|
||||
|
|
@ -191,20 +88,97 @@ module('Unit | Route | vault/cluster/oidc-callback', function (hooks) {
|
|||
a namespace-less state and authentication fails
|
||||
{ state: 'st_yOarDguU848w5YZuotLs,ns' }
|
||||
*/
|
||||
skip('it uses namespace when state param is not uri encoded', async function (assert) {
|
||||
this.routeName = 'vault.cluster.oidc-callback';
|
||||
this.callbackUrlQueryParams(`${this.state},ns=admin`);
|
||||
this.route.afterModel();
|
||||
assert.propEqual(
|
||||
this.windowStub.lastCall.args[0],
|
||||
{
|
||||
code: this.code,
|
||||
namespace: 'admin',
|
||||
path: this.path,
|
||||
test('it parses params correctly with regular inputs and namespace in state (unencoded)', function (assert) {
|
||||
const searchString = '?code=my-code&state=my-state,ns=foo/bar';
|
||||
const params = {
|
||||
state: 'my-state,ns', // Ember parses the QP incorrectly if unencoded
|
||||
code: 'my-code',
|
||||
path: 'oidc-path',
|
||||
};
|
||||
const results = getParamsForCallback(params, searchString);
|
||||
assert.deepEqual(results, {
|
||||
source: 'oidc-callback',
|
||||
state: this.state,
|
||||
},
|
||||
'namespace is parsed correctly'
|
||||
);
|
||||
...params,
|
||||
state: 'my-state',
|
||||
namespace: 'foo/bar',
|
||||
});
|
||||
});
|
||||
|
||||
test('it parses params correctly with regular inputs and namespace in state (encoded)', function (assert) {
|
||||
const qp = {
|
||||
state: 'my-state,ns=foo/bar', // Ember parses the QP correctly when encoded
|
||||
code: 'my-code',
|
||||
path: 'oidc-path',
|
||||
};
|
||||
const searchString = `?code=my-code&state=${encodeURIComponent(qp.state)}`;
|
||||
const results = getParamsForCallback(qp, searchString);
|
||||
assert.deepEqual(results, { source: 'oidc-callback', ...qp, state: 'my-state', namespace: 'foo/bar' });
|
||||
});
|
||||
|
||||
test('namespace in state takes precedence over namespace in route (encoded)', function (assert) {
|
||||
const qp = {
|
||||
state: 'my-state,ns=foo/bar',
|
||||
code: 'my-code',
|
||||
path: 'oidc-path',
|
||||
namespace: 'other/ns',
|
||||
};
|
||||
const searchString = `?code=my-code&state=${encodeURIComponent(
|
||||
qp.state
|
||||
)}&namespace=${encodeURIComponent(qp.namespace)}`;
|
||||
const results = getParamsForCallback(qp, searchString);
|
||||
assert.deepEqual(results, {
|
||||
source: 'oidc-callback',
|
||||
...qp,
|
||||
state: 'my-state',
|
||||
namespace: 'foo/bar',
|
||||
});
|
||||
});
|
||||
|
||||
test('namespace in state takes precedence over namespace in route (unencoded)', function (assert) {
|
||||
const qp = {
|
||||
state: 'my-state,ns',
|
||||
code: 'my-code',
|
||||
path: 'oidc-path',
|
||||
namespace: 'other/ns',
|
||||
};
|
||||
const searchString = `?code=${qp.code}&state=my-state,ns=foo/bar&namespace=${qp.namespace}`;
|
||||
const results = getParamsForCallback(qp, searchString);
|
||||
assert.deepEqual(results, {
|
||||
source: 'oidc-callback',
|
||||
...qp,
|
||||
state: 'my-state',
|
||||
namespace: 'foo/bar',
|
||||
});
|
||||
});
|
||||
|
||||
test('it parses params correctly when window.location.search is empty (HCP scenario)', function (assert) {
|
||||
const params = {
|
||||
state: 'some-state,ns=admin/child-ns',
|
||||
code: 'my-code',
|
||||
namespace: 'admin',
|
||||
path: 'oidc-path',
|
||||
};
|
||||
const results = getParamsForCallback(params, '');
|
||||
assert.deepEqual(results, {
|
||||
source: 'oidc-callback',
|
||||
code: 'my-code',
|
||||
path: 'oidc-path',
|
||||
state: 'some-state',
|
||||
namespace: 'admin/child-ns',
|
||||
});
|
||||
});
|
||||
|
||||
test('it defaults to reasonable values if query params are missing', function (assert) {
|
||||
const params = {
|
||||
path: 'oidc-path',
|
||||
};
|
||||
const results = getParamsForCallback(params, '');
|
||||
assert.deepEqual(results, {
|
||||
source: 'oidc-callback',
|
||||
code: '',
|
||||
path: 'oidc-path',
|
||||
state: '',
|
||||
});
|
||||
});
|
||||
});
|
||||
});
|
||||
|
|
|
|||
Loading…
Reference in New Issue