Skip to content
This repository was archived by the owner on Mar 12, 2025. It is now read-only.

Commit 64d29bd

Browse files
committed
fix: values of query params might be encoded when having special characters
Some query params may have special characters thus got encoded client-side. So I'm reverting code which uses "decodeURIComponent" to decode such query params. TODO: we might replace custom logic for decoding **some** query params with an automatic mechanism to decode **all** query params.
1 parent 3793765 commit 64d29bd

File tree

4 files changed

+5
-5
lines changed

4 files changed

+5
-5
lines changed

src/routes/milestoneTemplates/list.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ module.exports = [
1414
permissions('milestoneTemplate.view'),
1515
(req, res, next) => {
1616
// Parse the sort query
17-
let sort = req.query.sort ? req.query.sort : 'order';
17+
let sort = req.query.sort ? decodeURIComponent(req.query.sort) : 'order';
1818
if (sort && sort.indexOf(' ') === -1) {
1919
sort += ' asc';
2020
}

src/routes/milestones/list.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ module.exports = [
2828
permissions('milestone.view'),
2929
(req, res, next) => {
3030
// Parse the sort query
31-
let sort = req.query.sort ? req.query.sort : 'order';
31+
let sort = req.query.sort ? decodeURIComponent(req.query.sort) : 'order';
3232
if (sort && sort.indexOf(' ') === -1) {
3333
sort += ' asc';
3434
}

src/routes/phases/list.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ module.exports = [
2020
const projectId = _.parseInt(req.params.projectId);
2121

2222
// Parse the fields string to determine what fields are to be returned
23-
let fields = req.query.fields ? req.query.fields.split(',') : PHASE_ATTRIBUTES;
24-
let sort = req.query.sort ? req.query.sort : 'startDate';
23+
let fields = req.query.fields ? decodeURIComponent(req.query.fields).split(',') : PHASE_ATTRIBUTES;
24+
let sort = req.query.sort ? decodeURIComponent(req.query.sort) : 'startDate';
2525
if (sort && sort.indexOf(' ') === -1) {
2626
sort += ' asc';
2727
}

src/routes/projects/list.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -531,7 +531,7 @@ module.exports = [
531531
// handle filters
532532
let filters = _.omit(req.query, 'sort', 'perPage', 'page', 'fields');
533533

534-
let sort = req.query.sort ? req.query.sort : 'createdAt';
534+
let sort = req.query.sort ? decodeURIComponent(req.query.sort) : 'createdAt';
535535
if (sort && sort.indexOf(' ') === -1) {
536536
sort += ' asc';
537537
}

0 commit comments

Comments
 (0)