azures04/crafatar
1
0
Fork 0
mirror of https://github.com/azures04/crafatar.git synced 2026-05-06 11:00:39 +02:00
Code Issues Packages Projects Releases Wiki Activity

downstream caches shouldn't cache server errors, fixes #136

Browse Source
This commit is contained in:
jomo
2016-01-20 02:00:10 +01:00
parent e7242ce773
commit 6a5967dfba
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/response.js
View File

@@ -4,7 +4,7 @@ var crc = require("crc").crc32;
var human_status = { var human_status = {
"-2": "user error", // e.g. invalid size "-2": "user error", // e.g. invalid size
"-1": "server error", // e.g. network issues "-1": "server error", // e.g. mojang/network issues
0: "none", // cached as null (user has no skin) 0: "none", // cached as null (user has no skin)
1: "cached", // found on disk 1: "cached", // found on disk
2: "downloaded", // profile downloaded, skin downloaded from mojang servers 2: "downloaded", // profile downloaded, skin downloaded from mojang servers
@@ -85,6 +85,8 @@ module.exports = function(request, response, result) {
if (result.status === -2) { if (result.status === -2) {
response.writeHead(result.code || 422, headers); response.writeHead(result.code || 422, headers);
} else if (result.status === -1) { } else if (result.status === -1) {
// 500 responses shouldn't be cached
headers["Cache-Control"] = "private, max-age=0, no-cache";
response.writeHead(500, headers); response.writeHead(500, headers);
} else { } else {
if (result.body) { if (result.body) {