fix: ok statusCode can be 200..299 (#7919)

discordjs · May 14, 2022 · d1504f2 · d1504f2
1 parent b1a3aa9
commit d1504f2
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/packages/rest/__tests__/REST.test.ts b/packages/rest/__tests__/REST.test.ts
@@ -253,6 +253,21 @@ test('postEcho', async () => {
 	expect(await api.post('/postEcho', { body: { foo: 'bar' } })).toStrictEqual({ foo: 'bar' });
 });
 
+test('201 status code', async () => {
+	mockPool
+		.intercept({
+			path: genPath('/postNon200StatusCode'),
+			method: 'POST',
+		})
+		.reply((t) => ({
+			data: t.body!,
+			statusCode: 201,
+			responseOptions,
+		}));
+
+	expect(await api.post('/postNon200StatusCode', { body: { foo: 'bar' } })).toStrictEqual({ foo: 'bar' });
+});
+
 test('Old Message Delete Edge-Case: Old message', async () => {
 	mockPool
 		.intercept({

diff --git a/packages/rest/src/lib/handlers/SequentialHandler.ts b/packages/rest/src/lib/handlers/SequentialHandler.ts
@@ -391,7 +391,7 @@ export class SequentialHandler implements IHandler {
 			}
 		}
 
-		if (status === 200) {
+		if (status >= 200 && status < 300) {
 			return parseResponse(res);
 		} else if (status === 429) {
 			// A rate limit was hit - this may happen if the route isn't associated with an official bucket hash yet, or when first globally rate limited