Merge pull request #754 from storyblok/fix/retry-mechanism-bug

fix: update the retry function to check the response status correctly

src/index.ts

@@ -592,18 +592,6 @@ class Storyblok {
 		return new Promise(async (resolve, reject) => {
 			try {
 				const res = await this.throttle('get', url, params)
-				if (res.status === 429) {
-					retries = retries ? retries + 1 : 0
-
-					if (retries < this.maxRetries) {
-						console.log(`Hit rate limit. Retrying in ${retries} seconds.`)
-						await this.helpers.delay(1000 * retries)
-						return this.cacheResponse(url, params, retries)
-							.then(resolve)
-							.catch(reject)
-					}
-				}
-
 				if (res.status !== 200) {
 					return reject(res)
 				}
@@ -640,6 +628,17 @@ class Storyblok {
 
 				return resolve(response)
 			} catch (error: Error | any) {
+				if (error.response && error.status === 429) {
+					retries = retries ? retries + 1 : 0
+
+					if (retries < this.maxRetries) {
+						console.log(`Hit rate limit. Retrying in ${retries} seconds.`)
+						await this.helpers.delay(1000 * retries)
+						return this.cacheResponse(url, params, retries)
+							.then(resolve)
+							.catch(reject)
+					}
+				}
 				reject(error)
 			}
 		})