sureshanaparti commented on PR #12584: URL: https://github.com/apache/cloudstack/pull/12584#issuecomment-3847967589
> clgtm, but I have questions about the functionality. As this fixes a bug atm (NPE when tried) I think we can go ahead. Users/operators will have to define how a broken or removed link of an authenticator should be handled. E.G. invalidate/regenerate PW or disable account. I could imagine this should be configurable in which case we can revert this condition. > > We can also remove the need to be a native account now, and fix the NPE. > > < @sureshanaparti > updated @DaanHoogland it also checks for admin, domain admin account or the same user to change the password. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
