diff --git a/lib/networking.js b/lib/networking.js index 80f43da..c2f9253 100644 --- a/lib/networking.js +++ b/lib/networking.js @@ -88,16 +88,12 @@ exp.get_from_options = function(rid, url, options, callback) { break; case 404: case 204: + case 429: // this shouldn't usually happen, but occasionally does case 500: case 503: // we don't want to cache this body = null; break; - case 429: - // this shouldn't usually happen, but occasionally does - // forcing error so it's not cached - error = error || new Error("TooManyRequestsException"); - break; default: if (!error) { // Probably 500 or the likes diff --git a/lib/response.js b/lib/response.js index 143967c..8403ded 100644 --- a/lib/response.js +++ b/lib/response.js @@ -13,7 +13,7 @@ var human_status = { // print these, but without stacktrace -var silent_errors = ["ETIMEDOUT", "ESOCKETTIMEDOUT", "ECONNRESET", "EHOSTUNREACH", "ECONNREFUSED", "TooManyRequestsException"]; +var silent_errors = ["ETIMEDOUT", "ESOCKETTIMEDOUT", "ECONNRESET", "EHOSTUNREACH", "ECONNREFUSED"]; // handles HTTP responses // +request+ a http.IncomingMessage diff --git a/test/test.js b/test/test.js index a77879f..73c3bda 100644 --- a/test/test.js +++ b/test/test.js @@ -1013,8 +1013,8 @@ describe("Crafatar", function() { it("uuid should be rate limited", function(done) { networking.get_profile(rid, id, function() { networking.get_profile(rid, id, function(err, profile) { - assert.strictEqual(err.message, "TooManyRequestsException"); - assert.strictEqual(profile.error, "TooManyRequestsException"); + assert.ifError(err); + assert.strictEqual(profile, null); done(); }); });