mirror of
https://github.com/xcat2/confluent.git
synced 2026-09-02 15:36:05 +00:00
Markus Hilger
bd5d7ffb94
Redfish identifies an account by a string, and an implementation is free to use the account name, which this one does. The handler converted the last element of the path to an integer, so every per user read, update and delete answered "invalid literal for int() with base 10: 'root'" as an unexpected error, with a traceback to match. Confluent offered the id itself, listing the account as "root", and then could not accept it back. Take the element as given. Everything below already compares ids as strings, and the input parsing already keeps a non numeric uid, so only this conversion stood in the way. nodebmcpassword goes through exactly this path, reading users/all for the id and then writing to that account, so it could not work at all on such a bmc. ipmi users really are numbered slots, so the conversion is right there and stays, but say so when it fails rather than letting a ValueError surface as an unexpected error.