tree 1b7544710f1a3e41857802549222dc9b628fc196
parent a974c132c8eaa591acaa92019f9b0088b4815ff1
author Jonathan Doman <jonathan.doman@intel.com> 1692397674 -0700
committer Ed Tanous <ed@tanous.net> 1698427254 +0000

Refactor populateUserInfo

- No need to set error code in asyncResp since caller already does that.
  Then we can remove the asyncResp param altogether.
- Check if session is valid before unpacking properties to avoid
  unnecessary work.
- Use std::optional instead of pointers for slighter cleaner code.
- Enforce required properties for local users based on D-Bus interface
  documentation (UserGroups must be provided for local users).

Change-Id: I770d3556a0d62182b6abd72bfa3f8d62e2a105d1
Signed-off-by: Jonathan Doman <jonathan.doman@intel.com>