mirror of
https://github.com/ansible/awx.git
synced 2026-05-23 08:37:48 -02:30
Removes application requirement when creating a token
This commit is contained in:
@@ -28,7 +28,8 @@ function TokensStrings (BaseString) {
|
|||||||
DELETE_ACTION_LABEL: t.s('DELETE'),
|
DELETE_ACTION_LABEL: t.s('DELETE'),
|
||||||
SCOPE_PLACEHOLDER: t.s('Select a scope'),
|
SCOPE_PLACEHOLDER: t.s('Select a scope'),
|
||||||
SCOPE_READ_LABEL: t.s('Read'),
|
SCOPE_READ_LABEL: t.s('Read'),
|
||||||
SCOPE_WRITE_LABEL: t.s('Write')
|
SCOPE_WRITE_LABEL: t.s('Write'),
|
||||||
|
APPLICATION_HELP_TEXT: t.s('Leaving this field blank will result in the creation of a Personal Access Token which is not linked to an Application.')
|
||||||
};
|
};
|
||||||
|
|
||||||
ns.list = {
|
ns.list = {
|
||||||
|
|||||||
@@ -1,59 +1,72 @@
|
|||||||
function AddTokensController (
|
function AddTokensController (
|
||||||
models, $state, strings, Rest, Alert, Wait, GetBasePath,
|
models, $state, strings, Alert, Wait,
|
||||||
$filter, ProcessErrors, $scope
|
$filter, ProcessErrors, $scope
|
||||||
) {
|
) {
|
||||||
const vm = this || {};
|
const vm = this || {};
|
||||||
const { application } = models;
|
const { application, token, user } = models;
|
||||||
|
|
||||||
vm.mode = 'add';
|
vm.mode = 'add';
|
||||||
vm.strings = strings;
|
vm.strings = strings;
|
||||||
vm.panelTitle = strings.get('add.PANEL_TITLE');
|
vm.panelTitle = strings.get('add.PANEL_TITLE');
|
||||||
|
|
||||||
vm.form = {};
|
vm.form = {
|
||||||
|
application: {
|
||||||
vm.form.application = {
|
type: 'field',
|
||||||
type: 'field',
|
label: 'Application',
|
||||||
label: 'Application',
|
id: 'application',
|
||||||
id: 'application'
|
required: false,
|
||||||
};
|
help_text: strings.get('add.APPLICATION_HELP_TEXT'),
|
||||||
vm.form.description = {
|
_resource: 'application',
|
||||||
type: 'String',
|
_route: 'users.edit.tokens.add.application',
|
||||||
label: 'Description',
|
_model: application,
|
||||||
id: 'description'
|
_placeholder: strings.get('add.APP_PLACEHOLDER')
|
||||||
};
|
},
|
||||||
|
description: {
|
||||||
vm.form.application._resource = 'application';
|
type: 'String',
|
||||||
vm.form.application._route = 'users.edit.tokens.add.application';
|
label: 'Description',
|
||||||
vm.form.application._model = application;
|
id: 'description',
|
||||||
vm.form.application._placeholder = strings.get('add.APP_PLACEHOLDER');
|
required: false
|
||||||
vm.form.application.required = true;
|
},
|
||||||
|
scope: {
|
||||||
vm.form.description.required = false;
|
choices: [
|
||||||
|
[null, ''],
|
||||||
vm.form.scope = {
|
['read', strings.get('add.SCOPE_READ_LABEL')],
|
||||||
choices: [
|
['write', strings.get('add.SCOPE_WRITE_LABEL')]
|
||||||
[null, ''],
|
],
|
||||||
['read', strings.get('add.SCOPE_READ_LABEL')],
|
help_text: strings.get('add.SCOPE_HELP_TEXT'),
|
||||||
['write', strings.get('add.SCOPE_WRITE_LABEL')]
|
id: 'scope',
|
||||||
],
|
label: 'Scope',
|
||||||
help_text: strings.get('add.SCOPE_HELP_TEXT'),
|
required: true,
|
||||||
id: 'scope',
|
_component: 'at-input-select',
|
||||||
label: 'Scope',
|
_data: [
|
||||||
required: true,
|
[null, ''],
|
||||||
_component: 'at-input-select',
|
['read', strings.get('add.SCOPE_READ_LABEL')],
|
||||||
_data: [
|
['write', strings.get('add.SCOPE_WRITE_LABEL')]
|
||||||
[null, ''],
|
],
|
||||||
['read', strings.get('add.SCOPE_READ_LABEL')],
|
_exp: 'choice[1] for (index, choice) in state._data',
|
||||||
['write', strings.get('add.SCOPE_WRITE_LABEL')]
|
_format: 'selectFromOptions'
|
||||||
],
|
}
|
||||||
_exp: 'choice[1] for (index, choice) in state._data',
|
|
||||||
_format: 'selectFromOptions'
|
|
||||||
};
|
};
|
||||||
|
|
||||||
vm.form.save = payload => {
|
vm.form.save = payload => {
|
||||||
Rest.setUrl(`${GetBasePath('users')}${$state.params.user_id}/authorized_tokens`);
|
const postToken = _.has(payload, 'application') ?
|
||||||
return Rest.post(payload)
|
user.postAuthorizedTokens({
|
||||||
|
id: $state.params.user_id,
|
||||||
|
payload
|
||||||
|
}) : token.request('post', { data: payload });
|
||||||
|
|
||||||
|
return postToken
|
||||||
.then(({ data }) => {
|
.then(({ data }) => {
|
||||||
|
const refreshHTML = data.refresh_token ?
|
||||||
|
`<div class="TokenModal">
|
||||||
|
<div class="TokenModal-label">
|
||||||
|
${strings.get('add.REFRESH_TOKEN_LABEL')}
|
||||||
|
</div>
|
||||||
|
<div class="TokenModal-value">
|
||||||
|
${data.refresh_token}
|
||||||
|
</div>
|
||||||
|
</div>` : '';
|
||||||
|
|
||||||
Alert(strings.get('add.TOKEN_MODAL_HEADER'), `
|
Alert(strings.get('add.TOKEN_MODAL_HEADER'), `
|
||||||
<div class="TokenModal">
|
<div class="TokenModal">
|
||||||
<div class="TokenModal-label">
|
<div class="TokenModal-label">
|
||||||
@@ -63,14 +76,7 @@ function AddTokensController (
|
|||||||
${data.token}
|
${data.token}
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
<div class="TokenModal">
|
${refreshHTML}
|
||||||
<div class="TokenModal-label">
|
|
||||||
${strings.get('add.REFRESH_TOKEN_LABEL')}
|
|
||||||
</div>
|
|
||||||
<div class="TokenModal-value">
|
|
||||||
${data.refresh_token}
|
|
||||||
</div>
|
|
||||||
</div>
|
|
||||||
<div class="TokenModal">
|
<div class="TokenModal">
|
||||||
<div class="TokenModal-label">
|
<div class="TokenModal-label">
|
||||||
${strings.get('add.TOKEN_EXPIRES_LABEL')}
|
${strings.get('add.TOKEN_EXPIRES_LABEL')}
|
||||||
@@ -106,10 +112,8 @@ AddTokensController.$inject = [
|
|||||||
'resolvedModels',
|
'resolvedModels',
|
||||||
'$state',
|
'$state',
|
||||||
'TokensStrings',
|
'TokensStrings',
|
||||||
'Rest',
|
|
||||||
'Alert',
|
'Alert',
|
||||||
'Wait',
|
'Wait',
|
||||||
'GetBasePath',
|
|
||||||
'$filter',
|
'$filter',
|
||||||
'ProcessErrors',
|
'ProcessErrors',
|
||||||
'$scope'
|
'$scope'
|
||||||
|
|||||||
@@ -3,17 +3,21 @@ import AddController from './users-tokens-add.controller';
|
|||||||
|
|
||||||
const addTemplate = require('~features/users/tokens/users-tokens-add.partial.html');
|
const addTemplate = require('~features/users/tokens/users-tokens-add.partial.html');
|
||||||
|
|
||||||
function TokensDetailResolve ($q, Application) {
|
function TokensDetailResolve ($q, Application, Token, User) {
|
||||||
const promises = {};
|
const promises = {};
|
||||||
|
|
||||||
promises.application = new Application('options');
|
promises.application = new Application('options');
|
||||||
|
promises.token = new Token('options');
|
||||||
|
promises.user = new User('options');
|
||||||
|
|
||||||
return $q.all(promises);
|
return $q.all(promises);
|
||||||
}
|
}
|
||||||
|
|
||||||
TokensDetailResolve.$inject = [
|
TokensDetailResolve.$inject = [
|
||||||
'$q',
|
'$q',
|
||||||
'ApplicationModel'
|
'ApplicationModel',
|
||||||
|
'TokenModel',
|
||||||
|
'UserModel'
|
||||||
];
|
];
|
||||||
|
|
||||||
function isMeResolve ($rootScope, $stateParams, $state) {
|
function isMeResolve ($rootScope, $stateParams, $state) {
|
||||||
|
|||||||
26
awx/ui/client/lib/models/Token.js
Normal file
26
awx/ui/client/lib/models/Token.js
Normal file
@@ -0,0 +1,26 @@
|
|||||||
|
let Base;
|
||||||
|
|
||||||
|
function setDependentResources () {
|
||||||
|
this.dependentResources = [];
|
||||||
|
}
|
||||||
|
|
||||||
|
function TokenModel (method, resource, config) {
|
||||||
|
Base.call(this, 'tokens');
|
||||||
|
|
||||||
|
this.Constructor = TokenModel;
|
||||||
|
this.setDependentResources = setDependentResources.bind(this);
|
||||||
|
|
||||||
|
return this.create(method, resource, config);
|
||||||
|
}
|
||||||
|
|
||||||
|
function TokenModelLoader (BaseModel) {
|
||||||
|
Base = BaseModel;
|
||||||
|
|
||||||
|
return TokenModel;
|
||||||
|
}
|
||||||
|
|
||||||
|
TokenModelLoader.$inject = [
|
||||||
|
'BaseModel',
|
||||||
|
];
|
||||||
|
|
||||||
|
export default TokenModelLoader;
|
||||||
40
awx/ui/client/lib/models/User.js
Normal file
40
awx/ui/client/lib/models/User.js
Normal file
@@ -0,0 +1,40 @@
|
|||||||
|
let Base;
|
||||||
|
let $http;
|
||||||
|
|
||||||
|
function postAuthorizedTokens (params) {
|
||||||
|
const req = {
|
||||||
|
method: 'POST',
|
||||||
|
url: `${this.path}${params.id}/authorized_tokens/`
|
||||||
|
};
|
||||||
|
|
||||||
|
if (params.payload) {
|
||||||
|
req.data = params.payload;
|
||||||
|
}
|
||||||
|
|
||||||
|
return $http(req);
|
||||||
|
}
|
||||||
|
|
||||||
|
function UserModel (method, resource, config) {
|
||||||
|
Base.call(this, 'users');
|
||||||
|
|
||||||
|
this.Constructor = UserModel;
|
||||||
|
this.postAuthorizedTokens = postAuthorizedTokens.bind(this);
|
||||||
|
|
||||||
|
this.model.launch = {};
|
||||||
|
|
||||||
|
return this.create(method, resource, config);
|
||||||
|
}
|
||||||
|
|
||||||
|
function UserModelLoader (BaseModel, _$http_) {
|
||||||
|
Base = BaseModel;
|
||||||
|
$http = _$http_;
|
||||||
|
|
||||||
|
return UserModel;
|
||||||
|
}
|
||||||
|
|
||||||
|
UserModelLoader.$inject = [
|
||||||
|
'BaseModel',
|
||||||
|
'$http'
|
||||||
|
];
|
||||||
|
|
||||||
|
export default UserModelLoader;
|
||||||
@@ -22,11 +22,13 @@ import Project from '~models/Project';
|
|||||||
import Schedule from '~models/Schedule';
|
import Schedule from '~models/Schedule';
|
||||||
import ProjectUpdate from '~models/ProjectUpdate';
|
import ProjectUpdate from '~models/ProjectUpdate';
|
||||||
import SystemJob from '~models/SystemJob';
|
import SystemJob from '~models/SystemJob';
|
||||||
|
import Token from '~models/Token';
|
||||||
import UnifiedJobTemplate from '~models/UnifiedJobTemplate';
|
import UnifiedJobTemplate from '~models/UnifiedJobTemplate';
|
||||||
import WorkflowJob from '~models/WorkflowJob';
|
import WorkflowJob from '~models/WorkflowJob';
|
||||||
import WorkflowJobTemplate from '~models/WorkflowJobTemplate';
|
import WorkflowJobTemplate from '~models/WorkflowJobTemplate';
|
||||||
import WorkflowJobTemplateNode from '~models/WorkflowJobTemplateNode';
|
import WorkflowJobTemplateNode from '~models/WorkflowJobTemplateNode';
|
||||||
import UnifiedJob from '~models/UnifiedJob';
|
import UnifiedJob from '~models/UnifiedJob';
|
||||||
|
import User from '~models/User';
|
||||||
|
|
||||||
import ModelsStrings from '~models/models.strings';
|
import ModelsStrings from '~models/models.strings';
|
||||||
|
|
||||||
@@ -59,10 +61,12 @@ angular
|
|||||||
.service('UnifiedJobModel', UnifiedJob)
|
.service('UnifiedJobModel', UnifiedJob)
|
||||||
.service('ProjectUpdateModel', ProjectUpdate)
|
.service('ProjectUpdateModel', ProjectUpdate)
|
||||||
.service('SystemJobModel', SystemJob)
|
.service('SystemJobModel', SystemJob)
|
||||||
|
.service('TokenModel', Token)
|
||||||
.service('UnifiedJobTemplateModel', UnifiedJobTemplate)
|
.service('UnifiedJobTemplateModel', UnifiedJobTemplate)
|
||||||
.service('WorkflowJobModel', WorkflowJob)
|
.service('WorkflowJobModel', WorkflowJob)
|
||||||
.service('WorkflowJobTemplateModel', WorkflowJobTemplate)
|
.service('WorkflowJobTemplateModel', WorkflowJobTemplate)
|
||||||
.service('WorkflowJobTemplateNodeModel', WorkflowJobTemplateNode)
|
.service('WorkflowJobTemplateNodeModel', WorkflowJobTemplateNode)
|
||||||
|
.service('UserModel', User)
|
||||||
.service('ModelsStrings', ModelsStrings);
|
.service('ModelsStrings', ModelsStrings);
|
||||||
|
|
||||||
export default MODULE_NAME;
|
export default MODULE_NAME;
|
||||||
|
|||||||
Reference in New Issue
Block a user