From 38f88fc563bf7b1b9bf25ecce70c8b61b2608bf9 Mon Sep 17 00:00:00 2001
From: Julian Lam <julian@nodebb.org>
Date: Wed, 30 Sep 2020 11:57:40 -0400
Subject: [PATCH] fix: #8699 tags route is case sensitive, though tags are not

---
 src/controllers/tags.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/controllers/tags.js b/src/controllers/tags.js
index b73b6842b6f1..9191aa07c348 100644
--- a/src/controllers/tags.js
+++ b/src/controllers/tags.js
@@ -3,17 +3,19 @@
 const validator = require('validator');
 const nconf = require('nconf');
 
+const meta = require('../meta');
 const user = require('../user');
 const categories = require('../categories');
 const topics = require('../topics');
 const privileges = require('../privileges');
 const pagination = require('../pagination');
+const utils = require('../utils');
 const helpers = require('./helpers');
 
 const tagsController = module.exports;
 
 tagsController.getTag = async function (req, res) {
-	const tag = validator.escape(String(req.params.tag));
+	const tag = validator.escape(utils.cleanUpTag(req.params.tag, meta.config.maximumTagLength));
 	const page = parseInt(req.query.page, 10) || 1;
 
 	const templateData = {
@@ -27,8 +29,8 @@ tagsController.getTag = async function (req, res) {
 	const stop = start + settings.topicsPerPage - 1;
 	const states = [categories.watchStates.watching, categories.watchStates.notwatching, categories.watchStates.ignoring];
 	const [topicCount, tids, categoriesData] = await Promise.all([
-		topics.getTagTopicCount(req.params.tag),
-		topics.getTagTids(req.params.tag, start, stop),
+		topics.getTagTopicCount(tag),
+		topics.getTagTids(tag, start, stop),
 		helpers.getCategoriesByStates(req.uid, '', states),
 	]);