Hi Dolph, Thank you for your quick feedback,
2015-12-09 12:33 GMT+09:00 Dolph Mathews <dolph.math...@gmail.com>: > I just noticed you suggested "<verb> <subject>"; if that's the prevailing > form, then I'd suggest "Change password (self-service)". Nice title for the api-site. Now Tempest's method is update_user_own_password() which calls this API. That is still good since your explanation. Thanks Ken Ohmichi --- > On Tue, Dec 8, 2015 at 9:31 PM, Dolph Mathews <dolph.math...@gmail.com> > wrote: >> >> This is implemented as a "self-service user password change" on the v2 >> public API. The user is required to have a token for the password they are >> changing, and is required to know the original password before a new one can >> be set. >> >> There is a similar "administrative password reset" call on the v2 admin >> API (contrib.admin_crud) which does not require the original password: >> >> >> https://github.com/openstack/keystone/blob/master/keystone/contrib/admin_crud/core.py#L104-L114 >> >> On Tue, Dec 8, 2015 at 8:56 PM, Ken'ichi Ohmichi <ken1ohmi...@gmail.com> >> wrote: >>> >>> Hi Keystone-team, >>> >>> TL;DR: >>> Keystone v2 API supports PATCH on '/OS-KSCRUD/users/{user_id}', >>> What should we name the operation for? >>> >>> We are adding the api-site link to Tempest docstring for explaining >>> how to use each REST APIs of each project. >>> This work comes from previous discussion on openstack-dev[1]. >>> We can find problems on Tempest and the api-site and solve them, that >>> is good thing for improving the quality. >>> During this work[2], we are facing a problem difficult to distinguish >>> what title is the best for OS-KSCRUD extension on Keystone v2 API. >>> >>> Keystone v2 API provides an API on "OS-KSCRUD/users/<user-id>" with >>> PATCH. >>> That seems "Update user" from REST design, but according to the >>> following Keystone implementation, that seems "Set user password". >>> >>> keystone/contrib/user_crud/core.py >>> 131 mapper.connect('/OS-KSCRUD/users/{user_id}', >>> 132 controller=user_controller, >>> 133 action='set_user_password', <HERE> >>> 134 conditions=dict(method=['PATCH'])) >>> >>> Which is the best for this API? >>> >>> The api-site doesn't contain this API description[3], so it will be >>> great to get feedback for improving both the api-site and Tempest. >>> >>> Thanks >>> Ken Ohmichi >>> >>> --- >>> [1]: >>> http://lists.openstack.org/pipermail/openstack-dev/2015-July/068864.html >>> [2]: https://review.openstack.org/#/c/253806/ >>> [3]: https://bugs.launchpad.net/openstack-api-site/+bug/1524153 >>> >>> >>> __________________________________________________________________________ >>> OpenStack Development Mailing List (not for usage questions) >>> Unsubscribe: >>> openstack-dev-requ...@lists.openstack.org?subject:unsubscribe >>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev >> >> > > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
