commit 19fb6e719849aea2f0cafdee64cd7da8a7a2b67b
author Ratan Gupta <ratagupt@linux.vnet.ibm.com> Mon Mar 04 13:30:50 2019 +0530
committer Ed Tanous <ed.tanous@intel.com> Tue Mar 12 23:26:47 2019 +0000
tree 06807c999ef65f2eea811d5cd48bfae56e10de04
parent f65af9e8c0d28a4b725c0d05ecbeb3007e2a09cf

Redfish(AccountService): Send the correct error message for the read only property

MinPasswordLength and MaxPasswordLength is a readOnly property
in the account service schema, so send the correct message
if the user tries to PATCH these properties.

Change-Id: If231ca13ef97fc05928435ac14a8190a186beb06
Signed-off-by: Ratan Gupta <ratagupt@linux.vnet.ibm.com>

redfish-core/lib/account_service.hpp

diff --git a/redfish-core/lib/account_service.hpp b/redfish-core/lib/account_service.hpp
index 045f40c..1b7b4b8 100644
--- a/redfish-core/lib/account_service.hpp
+++ b/redfish-core/lib/account_service.hpp
@@ -167,12 +167,28 @@
 
         std::optional<uint32_t> unlockTimeout;
         std::optional<uint16_t> lockoutThreshold;
+        std::optional<uint16_t> minPasswordLength;
+        std::optional<uint16_t> maxPasswordLength;
+
         if (!json_util::readJson(req, res, "AccountLockoutDuration",
                                  unlockTimeout, "AccountLockoutThreshold",
-                                 lockoutThreshold))
+                                 lockoutThreshold, "MaxPasswordLength",
+                                 maxPasswordLength, "MinPasswordLength",
+                                 minPasswordLength))
         {
             return;
         }
+
+        if (minPasswordLength)
+        {
+            messages::propertyNotWritable(asyncResp->res, "MinPasswordLength");
+        }
+
+        if (maxPasswordLength)
+        {
+            messages::propertyNotWritable(asyncResp->res, "MaxPasswordLength");
+        }
+
         if (unlockTimeout)
         {
             crow::connections::systemBus->async_method_call(