Don't allow non-utf8 seaches

momijizukamori · momijizukamori · commit 1ed419ceffa7 · 2025-04-02T13:26:12.000-04:00
diff --git a/app/Http/Requests/Api/SearchRequest.php b/app/Http/Requests/Api/SearchRequest.php
@@ -58,6 +58,9 @@ protected function prepareForValidation()
         if (empty($this->sort) || !valid_sort($this->sort)) {
             $this->merge(['sort' => 'added_new']);
         }
+        if (!mb_detect_encoding($this->search, 'utf-8', true)) {
+            $this->merge(['search' => null]);
+        }
     }
 
 
diff --git a/app/Http/Requests/SearchRequest.php b/app/Http/Requests/SearchRequest.php
@@ -49,6 +49,9 @@ protected function prepareForValidation()
         if (empty($this->sort) || !valid_sort($this->sort)) {
             $this->merge(['sort' => 'added_new']);
         }
+        if (!mb_detect_encoding($this->search, 'utf-8', true)) {
+            $this->merge(['search' => null]);
+        }
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,9 @@ protected function prepareForValidation()`
`58`	`58`	`if (empty($this->sort) \|\| !valid_sort($this->sort)) {`
`59`	`59`	`$this->merge(['sort' => 'added_new']);`
`60`	`60`	`}`
	`61`	`+ if (!mb_detect_encoding($this->search, 'utf-8', true)) {`
	`62`	`+ $this->merge(['search' => null]);`
	`63`	`+ }`
`61`	`64`	`}`
`62`	`65`
`63`	`66`
Original file line number	Diff line number	Diff line change
`@@ -49,6 +49,9 @@ protected function prepareForValidation()`
`49`	`49`	`if (empty($this->sort) \|\| !valid_sort($this->sort)) {`
`50`	`50`	`$this->merge(['sort' => 'added_new']);`
`51`	`51`	`}`
	`52`	`+ if (!mb_detect_encoding($this->search, 'utf-8', true)) {`
	`53`	`+ $this->merge(['search' => null]);`
	`54`	`+ }`
`52`	`55`	`}`
`53`	`56`
`54`	`57`	`/**`