From 9faeedc03f19a0b784430ed2b2259e85db3b6d3d Mon Sep 17 00:00:00 2001 From: jomo Date: Wed, 10 Dec 2014 18:42:19 +0100 Subject: [PATCH] convert keys to lower case at cache level --- modules/cache.js | 4 ++++ routes/avatars.js | 2 +- routes/skins.js | 2 +- 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/modules/cache.js b/modules/cache.js index 1537c71..885c078 100644 --- a/modules/cache.js +++ b/modules/cache.js @@ -59,6 +59,8 @@ exp.get_redis = function() { exp.update_timestamp = function(uuid, hash) { logging.log(uuid + " cache: updating timestamp"); var time = new Date().getTime(); + // store uuid in lower case if not null + uuid = uuid && uuid.toLowerCase(); redis.hmset(uuid, "t", time); update_file_date(hash); }; @@ -78,6 +80,8 @@ exp.save_hash = function(uuid, hash) { // {hash: "0123456789abcdef", time: 1414881524512} // null when uuid unkown exp.get_details = function(uuid, callback) { + // get uuid in lower case if not null + uuid = uuid && uuid.toLowerCase(); redis.hgetall(uuid, function(err, data) { var details = null; if (data) { diff --git a/routes/avatars.js b/routes/avatars.js index 93f0d65..6537b59 100644 --- a/routes/avatars.js +++ b/routes/avatars.js @@ -15,7 +15,7 @@ var human_status = { /* GET avatar request. */ router.get("/:uuid.:ext?", function(req, res) { - var uuid = (req.params.uuid || "").toLowerCase(); + var uuid = (req.params.uuid || ""); var size = parseInt(req.query.size) || config.default_size; var def = req.query.default; var helm = req.query.hasOwnProperty("helm"); diff --git a/routes/skins.js b/routes/skins.js index 1680cfa..4493847 100644 --- a/routes/skins.js +++ b/routes/skins.js @@ -8,7 +8,7 @@ var lwip = require("lwip"); /* GET skin request. */ router.get("/:uuid.:ext?", function(req, res) { - var uuid = (req.params.uuid || "").toLowerCase(); + var uuid = (req.params.uuid || ""); var def = req.query.default; var start = new Date(); var etag = null;