diff options
| author | Jonathan Doman <jonathan.doman@intel.com> | 2023-08-19 01:27:54 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2023-10-27 20:20:54 +0300 |
| commit | 522377dcb85082da598403e104a44d621b4c2bb4 (patch) | |
| tree | 1b7544710f1a3e41857802549222dc9b628fc196 /http/complete_response_fields.hpp | |
| parent | a974c132c8eaa591acaa92019f9b0088b4815ff1 (diff) | |
| download | bmcweb-522377dcb85082da598403e104a44d621b4c2bb4.tar.xz | |
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>
Diffstat (limited to 'http/complete_response_fields.hpp')
0 files changed, 0 insertions, 0 deletions