fix: #8929, fix popular, top rss feed urls

4 years ago · 77f0bff54f
parent 8f4060819f
commit 77f0bff54f
2 changed files with 2 additions and 2 deletions
--- a/src/controllers/popular.js
+++ b/src/controllers/popular.js
@ -22,7 +22,7 @@ popularController.get = async function (req, res, next) {
 	}

 	const feedQs = data.rssFeedUrl.split('?')[1];
-	data.rssFeedUrl = nconf.get('relative_path') + '/popular/' + (validator.escape(String(req.query.term)) || 'alltime') + '.rss';
+	data.rssFeedUrl = nconf.get('relative_path') + '/popular/' + validator.escape(String(req.query.term || 'alltime')) + '.rss';
 	if (req.loggedIn) {
 		data.rssFeedUrl += '?' + feedQs;
 	}
--- a/src/controllers/top.js
+++ b/src/controllers/top.js
@ -20,7 +20,7 @@ topController.get = async function (req, res, next) {
 	}

 	const feedQs = data.rssFeedUrl.split('?')[1];
-	data.rssFeedUrl = nconf.get('relative_path') + '/top/' + (validator.escape(String(req.query.term)) || 'alltime') + '.rss';
+	data.rssFeedUrl = nconf.get('relative_path') + '/top/' + validator.escape(String(req.query.term || 'alltime')) + '.rss';
 	if (req.loggedIn) {
 		data.rssFeedUrl += '?' + feedQs;
 	}