After creating application credential without selecting "Unrestricted
(dangerous)" checkbox,
it was confirmed that "no/いいえ" is shown in "Unrestricted" field as
expected.
Thank you very much.
Taka
On 2019年04月05日 10:18, Yuko Katabami wrote:
> I got the following information by an engineer who looked into the code.
> It seems that "None" should be changed to "False". It was tested and verified in devstack.
> Please take a look.
> Hope this is helpful!
>
>
> [root@devstack horizon]# git diff
> diff --git a/openstack_dashboard/dashboards/identity/application_credentials/forms.py b/openstack_dashboard/dashboards/identity/application_credentials/forms.py
> index 5b6286f..447524a 100644
> --- a/openstack_dashboard/dashboards/identity/application_credentials/forms.py
> +++ b/openstack_dashboard/dashboards/identity/application_credentials/forms.py
> @@ -94,7 +94,7 @@ class CreateApplicationCredentialForm(forms.SelfHandlingForm):
> secret=data['secret'] or None,
> expires_at=expiration or None,
> roles=roles,
> - unrestricted=data['unrestricted'] or None
> + unrestricted=data['unrestricted'] or False
> )
> self.request.session['application_credential'] = \
> new_app_cred.to_dict()
>
After creating application credential without selecting "Unrestricted
(dangerous)" checkbox,
it was confirmed that "no/いいえ" is shown in "Unrestricted" field as
expected.
Thank you very much.
Taka
On 2019年04月05日 10:18, Yuko Katabami wrote: dashboard/ dashboards/ identity/ application_ credentials/ forms.py b/openstack_ dashboard/ dashboards/ identity/ application_ credentials/ forms.py dashboard/ dashboards/ identity/ application_ credentials/ forms.py dashboard/ dashboards/ identity/ application_ credentials/ forms.py onCredentialFor m(forms. SelfHandlingFor m): data['secret' ] or None, at=expiration or None, data['unrestric ted'] or None data['unrestric ted'] or False session[ 'application_ credential' ] = \ cred.to_ dict()
> I got the following information by an engineer who looked into the code.
> It seems that "None" should be changed to "False". It was tested and verified in devstack.
> Please take a look.
> Hope this is helpful!
>
>
> [root@devstack horizon]# git diff
> diff --git a/openstack_
> index 5b6286f..447524a 100644
> --- a/openstack_
> +++ b/openstack_
> @@ -94,7 +94,7 @@ class CreateApplicati
> secret=
> expires_
> roles=roles,
> - unrestricted=
> + unrestricted=
> )
> self.request.
> new_app_
>