diff options
author | jmbills <jason.m.bills@intel.com> | 2022-01-27 01:16:17 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-01-27 01:16:17 +0300 |
commit | 46e83c05e9521fb20a8aa84c1534083f99650d5f (patch) | |
tree | 35670b94965a2e46fc79d89784cfb35cf7bfaf2f /meta-openbmc-mods/meta-common/recipes-phosphor/interfaces/bmcweb/0026-Fix-Query-Parameters-issue.patch | |
parent | 7cf0c1cd0ce835d1833509b7b911e8a97380278b (diff) | |
parent | 0eea29e45f1d671f0a769bb433cb6593593b7017 (diff) | |
download | openbmc-46e83c05e9521fb20a8aa84c1534083f99650d5f.tar.xz |
Merge pull request #77 from Intel-BMC/update1-0.86-71-0.86-postbeta
Update to internal 1-0.86-7
Diffstat (limited to 'meta-openbmc-mods/meta-common/recipes-phosphor/interfaces/bmcweb/0026-Fix-Query-Parameters-issue.patch')
-rw-r--r-- | meta-openbmc-mods/meta-common/recipes-phosphor/interfaces/bmcweb/0026-Fix-Query-Parameters-issue.patch | 44 |
1 files changed, 44 insertions, 0 deletions
diff --git a/meta-openbmc-mods/meta-common/recipes-phosphor/interfaces/bmcweb/0026-Fix-Query-Parameters-issue.patch b/meta-openbmc-mods/meta-common/recipes-phosphor/interfaces/bmcweb/0026-Fix-Query-Parameters-issue.patch new file mode 100644 index 000000000..d4c629c5a --- /dev/null +++ b/meta-openbmc-mods/meta-common/recipes-phosphor/interfaces/bmcweb/0026-Fix-Query-Parameters-issue.patch @@ -0,0 +1,44 @@ +From e40f454d5784570d497449775ae9f77f77973353 Mon Sep 17 00:00:00 2001 +From: "Arun P. Mohanan" <arun.p.m@linux.intel.com> +Date: Sat, 4 Dec 2021 01:28:38 +0530 +Subject: [PATCH] Fix Query Parameters issue + +Due to upstream change https://github.com/openbmc/bmcweb/commit/59b98b2222fddbea3d6f678d9e94006521f0c381, +none of the query parameters are functional in bmcweb. + +This commit adds the code required to copy the query parameters to +the crow request which is the actual request object used for handling +incoming URI requests. + +Tested: +GET on https://<BMC_IP>/redfish/v1/Systems/system/LogServices/EventLog/Entries +and https://<BMC_IP>/redfish/v1/Systems/system/LogServices/EventLog/Entries?skip=1000 + +Gave different logs indication correct functionality of skip query +parameter + +https://<BMC_IP>/redfish/v1/EventService/Subscriptions/SSE?$filter=(MessageId%20eq%20SELEntryAddedsaf) +Succesfully returned error indicating correct functionality of +query parameters as SELEntryAddedsaf is an invalid message id. + +Signed-off-by: Arun P. Mohanan <arun.p.m@linux.intel.com> +Signed-off-by: P Dheeraj Srujan Kumar <p.dheeraj.srujan.kumar@intel.com> +--- + http/http_connection.hpp | 1 + + 1 file changed, 1 insertion(+) + +diff --git a/http/http_connection.hpp b/http/http_connection.hpp +index cdd3707..940b8b8 100644 +--- a/http/http_connection.hpp ++++ b/http/http_connection.hpp +@@ -339,6 +339,7 @@ class Connection : + try + { + thisReq.urlView = boost::urls::url_view(thisReq.target()); ++ thisReq.urlParams = thisReq.urlView.params(); + thisReq.url = thisReq.urlView.encoded_path(); + } + catch (std::exception& p) +-- +2.17.1 + |