Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

fix(api): Removing host type AND having the cursor in the search bar sends an error 500 #8432

Closed
wants to merge 2 commits into from
+9 −0
Closed

fix(api): Removing host type AND having the cursor in the search bar sends an error 500 #8432

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2757fbe
fix(api): prevent executing of SQL query with wrong REGEXP syntaxis
vhr Mar 19, 2020
4fe3baa
Update src/Centreon/Infrastructure/RequestParameters/SqlRequestParame…
vhr Mar 20, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions src/Centreon/Infrastructure/RequestParameters/SqlRequestParametersTranslator.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -200,6 +200,15 @@ private function createQueryOnKeyValue(string $key, $valueOrArray): string
if (is_array($valueOrArray)) {
$searchOperator = (string) key($valueOrArray);
$mixedValue = $valueOrArray[$searchOperator];

// replace REGEXP operator with LIKE if it's not valid
if (
$mixedValue
&& $searchOperator === RequestParameters::OPERATOR_REGEXP
&& @preg_match("/{$mixedValue}/", '') === false
) {
$searchOperator = RequestParameters::OPERATOR_LIKE;
}
} else {
$searchOperator = RequestParameters::DEFAULT_SEARCH_OPERATOR;
$mixedValue = $valueOrArray;
Expand Down