Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-11-02 12:05:57 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix double-URL-encoding causing redirect loops

Browse Source
This commit is contained in:
Ben Lubar
2016-03-22 19:54:27 -05:00
parent 89aa06526d
commit c7cfe45d0b
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/controllers/category.js
View File

@@ -52,7 +52,7 @@ categoryController.get = function(req, res, callback) {
} }
if (!res.locals.isAPI && (!req.params.slug || results.categoryData.slug !== cid + '/' + req.params.slug) && (results.categoryData.slug && results.categoryData.slug !== cid + '/')) { if (!res.locals.isAPI && (!req.params.slug || results.categoryData.slug !== cid + '/' + req.params.slug) && (results.categoryData.slug && results.categoryData.slug !== cid + '/')) {
return helpers.redirect(res, '/category/' + encodeURI(results.categoryData.slug)); return helpers.redirect(res, '/category/' + results.categoryData.slug);
} }
var settings = results.userSettings; var settings = results.userSettings;

2
src/controllers/topics.js
View File

@@ -55,7 +55,7 @@ topicsController.get = function(req, res, callback) {
} }
if (!res.locals.isAPI && (!req.params.slug || results.topic.slug !== tid + '/' + req.params.slug) && (results.topic.slug && results.topic.slug !== tid + '/')) { if (!res.locals.isAPI && (!req.params.slug || results.topic.slug !== tid + '/' + req.params.slug) && (results.topic.slug && results.topic.slug !== tid + '/')) {
var url = '/topic/' + encodeURI(results.topic.slug); var url = '/topic/' + results.topic.slug;
if (req.params.post_index){ if (req.params.post_index){
url += '/'+req.params.post_index; url += '/'+req.params.post_index;
} }