Yes, basically you don't need to make any network calls here.

Thanks,
Bhathiya

On Mon, Jun 15, 2020 at 9:41 PM Samitha Chathuranga <sami...@wso2.com>
wrote:

> Hi Bhathiya,
>
>> Three basic steps implemented in the APIConsumerImpl for this
>>> changePassword operation are,
>>> 1. Validate current password
>>>
>>> 2. Check whether 'Password Change' is enabled by newly added
>>> configuration element into api-manager.xml as below.
>>>     <EnableChangePassword>true<EnableChangePassword/>
>>>
>>> (In the 2.6.0 implementation, the condition was to check whether
>>> self-signup was enabled for the particular tenant. But it is not ideal to
>>> decide the possibility to change the password based on this condition,
>>> since there are concerns as one would need to change the password, even
>>> when self-signup was disabled. So it was decided to use the above mentioned
>>> approach, so that the admins can decide this directly via a config)
>>>
>>> 3. Change the password by calling UserAdmin service via UserAdminStub (a
>>> new UserAdminClient class is implemented for this)
>>>
>>
>> We decided to do this via the OSGi service directly, right?
>>
>
> AFAIR we decided to use OSGI service directly for the 1st step (Validate
> current password) and for the 4th step (this 4th step was there in 2.x.x
> implementation and we decided to remove it).
> Anyway since this is a better approach, I will check for the possibility
> of using OSGI service for password change operation too.
>
> Thanks,
> Samitha
>
>>
>> Thanks,
>> Bhathiya
>>
>>
>>>
>>> The basic REST API and the implementation are completed ATM and works
>>> for super tenant users. Has to check further on the functionality for
>>> tenant users.
>>>
>>> TODO:
>>>
>>>    - Check further on the functionality for tenant users
>>>    - Implement the UI
>>>
>>> Appreciate any feedback and please add if I have missed anything.
>>>
>>> The related github issues for this effort are [1] (REST API
>>> Implementation) and [2] (UX implementation)
>>>
>>> [1] https://github.com/wso2/product-apim/issues/8181
>>> [2] https://github.com/wso2/product-apim/issues/8182
>>>
>>> Regards,
>>> Samitha
>>>
>>> --
>>> *Samitha Chathuranga*
>>> *Associate Technical Lead*, *WSO2 Inc.*
>>> lean.enterprise.middleware
>>> Mobile: +94715123761
>>>
>>> [image: http://wso2.com/signature] <http://wso2.com/signature>
>>>
>>
>>
>> --
>> *Bhathiya Jayasekara* | Senior Technical Lead | WSO2 Inc.
>> (m) +94 71 547 8185  | (e) bhathiya-@t-wso2-d0t-com
>>
>>
>>
>
> --
> *Samitha Chathuranga*
> *Associate Technical Lead*, *WSO2 Inc.*
> lean.enterprise.middleware
> Mobile: +94715123761
>
> [image: http://wso2.com/signature] <http://wso2.com/signature>
>


-- 
*Bhathiya Jayasekara* | Senior Technical Lead | WSO2 Inc.
(m) +94 71 547 8185  | (e) bhathiya-@t-wso2-d0t-com
_______________________________________________
Architecture mailing list
Architecture@wso2.org
https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture

Reply via email to