Mirrors/unleash.unleash
1
0
Fork 0
mirror of https://github.com/Unleash/unleash.git synced 2025-06-04 01:18:20 +02:00
Code Issues Projects Releases Wiki Activity

fix: not crash if addon http post throws (#738)

Browse Source 
added try/catch logic around fetchRetry function so that we do not
crash if addon fetch call throws
This commit is contained in:
Ivar Conradi Østhus 2021-02-26 13:46:56 +01:00 committed by GitHub
parent f9fd65a4bf
commit 05e69e6663
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 28 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
src/lib/addons/addon.js
View File

@ -28,16 +28,36 @@ class Addon {
async fetchRetry(url, options = {}, retries = 1, backoff = 300) { async fetchRetry(url, options = {}, retries = 1, backoff = 300) {
const retryCodes = [408, 500, 502, 503, 504, 522, 524]; const retryCodes = [408, 500, 502, 503, 504, 522, 524];
const res = await fetch(url, options); try {
if (res.ok) { const res = await fetch(url, options);
if (res.ok) {
return res;
}
if (retries > 0 && retryCodes.includes(res.status)) {
setTimeout(() => {
return this.fetchRetry(
url,
options,
retries - 1,
backoff * 2,
);
}, backoff);
}
return res; return res;
} catch (error) {
this.logger.warn(error);
if (retries > 0) {
setTimeout(() => {
return this.fetchRetry(
url,
options,
retries - 1,
backoff * 2,
);
}, backoff);
}
return { status: 500 };
} }
if (retries > 0 && retryCodes.includes(res.status)) {
setTimeout(() => {
return this.fetchRetry(url, options, retries - 1, backoff * 2);
}, backoff);
}
return res;
} }
} }