commit 1d40ef697d61199197214380ac01970b30d5d575
author Jayaprakash Mutyala <mutyalax.jayaprakash@intel.com> Fri Feb 26 13:06:03 2021 +0000
committer Jayaprakash Mutyala <mutyalax.jayaprakash@intel.com> Fri Mar 12 05:02:40 2021 +0000
tree 86d6c1f1ffa2dbe55c4737e713f2d91c800cf936
parent d32392249b1538e0b0a76f20bf602e683f636fb6

led: Fix for Chassis IndicatorLED patch response

While Patching Chassis Indicator LED from Redfish, response is showing
as empty but HTTPS status code is "200 OK" on successful case.
So provided fix for Proper response on Success.

Tested:
1. Verified RedFish validator passed
2. Verified the response on Redfish by updating IndicatorLED status.
PATCH: https://<BMC-IP>/redfish/v1/Chassis/<Baseboard-ID>
Body:
{
	"IndicatorLED": "Blinking"
}
Response:
{
    "@Message.ExtendedInfo": [
        {
            "@odata.type": "#Message.v1_1_1.Message",
            "Message": "Successfully Completed Request",
            "MessageArgs": [],
            "MessageId": "Base.1.8.1.Success",
            "MessageSeverity": "OK",
            "Resolution": "None"
        }
    ]
}

Signed-off-by: Jayaprakash Mutyala <mutyalax.jayaprakash@intel.com>
Change-Id: I2e1281e6c06c445fe8d0c350bb74ea18f3461a35

redfish-core/lib/led.hpp

diff --git a/redfish-core/lib/led.hpp b/redfish-core/lib/led.hpp
index 0fd1c9b..c003d22 100644
--- a/redfish-core/lib/led.hpp
+++ b/redfish-core/lib/led.hpp
@@ -141,6 +141,7 @@
                         messages::internalError(aResp->res);
                         return;
                     }
+                    messages::success(aResp->res);
                 },
                 "xyz.openbmc_project.LED.GroupManager",
                 "/xyz/openbmc_project/led/groups/enclosure_identify",