diff options
Diffstat (limited to 'redfish-core/include')
-rw-r--r-- | redfish-core/include/utils/query_param.hpp | 2 | ||||
-rw-r--r-- | redfish-core/include/utils/query_param_test.cpp | 14 |
2 files changed, 5 insertions, 11 deletions
diff --git a/redfish-core/include/utils/query_param.hpp b/redfish-core/include/utils/query_param.hpp index f51993ea90..45cb43bb48 100644 --- a/redfish-core/include/utils/query_param.hpp +++ b/redfish-core/include/utils/query_param.hpp @@ -431,7 +431,7 @@ inline std::optional<Query> return std::nullopt; } } - else if (key == "$select" && bmcwebInsecureEnableQueryParams) + else if (key == "$select") { if (!getSelectParam(value, ret)) { diff --git a/redfish-core/include/utils/query_param_test.cpp b/redfish-core/include/utils/query_param_test.cpp index 7358821ae2..965b6adf9c 100644 --- a/redfish-core/include/utils/query_param_test.cpp +++ b/redfish-core/include/utils/query_param_test.cpp @@ -358,16 +358,10 @@ TEST(RecursiveSelect, ReservedPropertiesAreSelected) ASSERT_TRUE(ret); crow::Response res; std::optional<Query> query = parseParameters(ret->params(), res); - if constexpr (bmcwebInsecureEnableQueryParams) - { - ASSERT_NE(query, std::nullopt); - recursiveSelect(root, query->selectTrie.root); - EXPECT_EQ(root, expected); - } - else - { - EXPECT_EQ(query, std::nullopt); - } + + ASSERT_NE(query, std::nullopt); + recursiveSelect(root, query->selectTrie.root); + EXPECT_EQ(root, expected); } TEST(QueryParams, ParseParametersOnly) |