opencomputeproject / HWMgmt-DeviceMgr-DeviceManager

Device Manager collects device data and notifications from each device, and make the data available on a predetermined output bus for consumers.
Apache License 2.0
23 stars 12 forks source link

ChangeDeviceUserPassword func does not work #13

Closed ehaligow closed 2 years ago

ehaligow commented 2 years ago

PATCH is sent to the wrong URL (it returns Not Found response). The URL used is wrongly created – instead of Id, username is used. The mentioned method searches in this collection the following URL:”/redfish/v1/AccountService/Accounts/root” for root user where “/redfish/v1/AccountService /Accounts/2” is assigned to root.

jcleung5549 commented 2 years ago

DM Mtg (Mar 8) - There is a race condition when the Redfish client GETS the Account list and then performs an Action on the one of the accounts, the account could have been deleted and recreated in the duration between the two HTTP actions. Do ETags solve this? Are ETags passed with with an POST Action? Should ODIM BMC simulate return an Etag per spec? Submitting issue to redfishforum.com.

JasonscHuang commented 2 years ago

Please do not test this version. We will provide the new code. You could test it later.

JasonscHuang commented 2 years ago

I already uploaded PR #15. Please see this issue has been fixed or not.

ehaligow commented 2 years ago

The issue is resolved.