Browse Source

convert uuid params to lower case, store only lower case uuids

jomo 10 years ago
parent
commit
e726fb9d5f
3 changed files with 4 additions and 2 deletions
  1. 2 0
      modules/cache.js
  2. 1 1
      routes/avatars.js
  3. 1 1
      routes/skins.js

+ 2 - 0
modules/cache.js

@@ -67,6 +67,8 @@ exp.update_timestamp = function(uuid, hash) {
 exp.save_hash = function(uuid, hash) {
   logging.log(uuid + " cache: saving hash");
   var time = new Date().getTime();
+  // store uuid in lower case if not null
+  uuid = uuid && uuid.toLowerCase();
   redis.hmset(uuid, "h", hash, "t", time);
 };
 

+ 1 - 1
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;
+  var uuid = (req.params.uuid || "").toLowerCase();
   var size = parseInt(req.query.size) || config.default_size;
   var def = req.query.default;
   var helm = req.query.hasOwnProperty("helm");

+ 1 - 1
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;
+  var uuid = (req.params.uuid || "").toLowerCase();
   var def = req.query.default;
   var start = new Date();
   var etag = null;