aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar Alexis Degrugillier <aledeg@users.noreply.github.com> 2021-03-19 08:51:42 -0400
committerGravatar GitHub <noreply@github.com> 2021-03-19 13:51:42 +0100
commit789a1664960da1b6287e3bb815d64e0d0fa8ef95 (patch)
treee8ae45b6f9c6b81bf986305fac5acae936cc79e7
parentb3340c32e38e7a4e39a9ac66e3d925386c202390 (diff)
Fix search param encoding in user query (#3541)
Before, the chosen encoding was not the best for the task since some part of the value was truncated when used with quotes. Now, the encoding allows to work smoothly with quotes. See #3538
-rwxr-xr-xapp/Controllers/configureController.php2
-rw-r--r--app/views/helpers/configure/query.phtml2
2 files changed, 2 insertions, 2 deletions
diff --git a/app/Controllers/configureController.php b/app/Controllers/configureController.php
index b2c1b5109..31619bf80 100755
--- a/app/Controllers/configureController.php
+++ b/app/Controllers/configureController.php
@@ -347,7 +347,7 @@ class FreshRSS_configure_Controller extends Minz_ActionController {
if (Minz_Request::isPost()) {
$params = array_filter(Minz_Request::param('query', []));
if (!empty($params['search'])) {
- $params['search'] = urldecode($params['search']);
+ $params['search'] = htmlspecialchars_decode($params['search'], ENT_QUOTES);
}
if (!empty($params['state'])) {
$params['state'] = array_sum($params['state']);
diff --git a/app/views/helpers/configure/query.phtml b/app/views/helpers/configure/query.phtml
index 7298a0a9a..38a1a715c 100644
--- a/app/views/helpers/configure/query.phtml
+++ b/app/views/helpers/configure/query.phtml
@@ -19,7 +19,7 @@
<div class="form-group">
<label class="group-name" for=""><?= _t('conf.query.filter.search') ?></label>
<div class="group-controls">
- <input type="text" id="query_search" name="query[search]" class="extend" value="<?= urldecode($this->query->getSearch()) ?>"/>
+ <input type="text" id="query_search" name="query[search]" class="extend" value="<?= htmlspecialchars($this->query->getSearch(), ENT_COMPAT, 'UTF-8') ?>"/>
</div>
</div>
<div class="form-group">