commit 927c17cd3a67d77517bf8ca7b779db858344ea15
author Radivoje Jovanovic <radivoje.jovanovic@intel.com> Mon Feb 14 08:10:47 2022 -0800
committer Radivoje (Ogi) Jovanovic <radivoje.jovanovic@intel.com> Thu Feb 24 16:37:18 2022 +0000
tree 743b88a14d50deb0a429ce7e9a9396778dfc35b9
parent 4978b63fc40588cd7beb52d94172585fdbb17fc7

Do not send duplicate data for ntp servers

in the case where multiple network interfaces are present each
interface will return the same ntp server data

Tested: on the system that has multiple eth interfaces

Signed-off-by: Radivoje Jovanovic <radivoje.jovanovic@intel.com>
Change-Id: I642aae7a1c07f7629d696f177ddf326d25e36fb1

redfish-core/lib/network_protocol.hpp

diff --git a/redfish-core/lib/network_protocol.hpp b/redfish-core/lib/network_protocol.hpp
index 7b665eb..eda9915 100644
--- a/redfish-core/lib/network_protocol.hpp
+++ b/redfish-core/lib/network_protocol.hpp
@@ -132,7 +132,7 @@
 
     getEthernetIfaceData([hostName, asyncResp](
                              const bool& success,
-                             const std::vector<std::string>& ntpServers,
+                             std::vector<std::string>& ntpServers,
                              const std::vector<std::string>& domainNames) {
         if (!success)
         {
@@ -140,6 +140,7 @@
                                        "NetworkProtocol");
             return;
         }
+        stl_utils::removeDuplicate(ntpServers);
         asyncResp->res.jsonValue["NTP"]["NTPServers"] = ntpServers;
         if (!hostName.empty())
         {