🐛 admin changepass should be able to respond with 404

This commit is contained in:
EntireTwix 2021-07-15 15:39:16 -07:00
parent 822a0d2647
commit 162b421c8b
2 changed files with 11 additions and 4 deletions

View file

@ -51,7 +51,7 @@
| name | 400 | 401 | 404 | 406 | | name | 400 | 401 | 404 | 406 |
| :------------------ | :----------------: | :----------------: | :----------------------: | :----------------: | | :------------------ | :----------------: | :----------------: | :----------------------: | :----------------: |
| ChangePassword | :heavy_check_mark: | :heavy_check_mark: | :heavy_multiplication_x: | :heavy_check_mark: | | ChangePassword | :heavy_check_mark: | :heavy_check_mark: | :heavy_multiplication_x: | :heavy_check_mark: |
| AdminChangePassword | :heavy_check_mark: | :heavy_check_mark: | :heavy_multiplication_x: | :heavy_check_mark: | | AdminChangePassword | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: |
| SetBal | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: | | SetBal | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: |
### Meta endpoint support ### Meta endpoint support

View file

@ -119,9 +119,16 @@ void api::AdminChangePassword(req_args) const
else else
{ {
StrFromSV_Wrapper name_val(name.value()); StrFromSV_Wrapper name_val(name.value());
StrFromSV_Wrapper pass_val(pass.value()); if (bank.Contains(name_val.str))
bank.ChangePassword(name_val.str, pass_val.str); {
res = BankResponse{k204NoContent, std::nullopt}; StrFromSV_Wrapper pass_val(pass.value());
bank.ChangePassword(name_val.str, pass_val.str);
res = BankResponse{k204NoContent, std::nullopt};
}
else
{
res = BankResponse{k404NotFound, "\"User not found\""};
}
} }
} }
RESPONSE_PARSE(std::move(res)); RESPONSE_PARSE(std::move(res));