Skip to content

Commit 5367228

Browse files
authoredNov 13, 2018
Remove unnecessary if block
1 parent 64f0adc commit 5367228

File tree

1 file changed

+16
-18
lines changed

1 file changed

+16
-18
lines changed
 

‎source/normalize-arguments.js

+16-18
Original file line numberDiff line numberDiff line change
@@ -210,29 +210,27 @@ const normalize = (url, options, defaults) => {
210210
return 0;
211211
}
212212

213-
if (error !== null) {
214-
if (!isRetryOnNetworkErrorAllowed(error) && (!options.retry.methods.has(error.method) || !options.retry.statusCodes.has(error.statusCode))) {
215-
return 0;
216-
}
217-
218-
if (Reflect.has(error, 'headers') && Reflect.has(error.headers, 'retry-after') && retryAfterStatusCodes.has(error.statusCode)) {
219-
let after = Number(error.headers['retry-after']);
220-
if (is.nan(after)) {
221-
after = Date.parse(error.headers['retry-after']) - Date.now();
222-
} else {
223-
after *= 1000;
224-
}
225-
226-
if (after > options.retry.maxRetryAfter) {
227-
return 0;
228-
}
213+
if (!isRetryOnNetworkErrorAllowed(error) && (!options.retry.methods.has(error.method) || !options.retry.statusCodes.has(error.statusCode))) {
214+
return 0;
215+
}
229216

230-
return after;
217+
if (Reflect.has(error, 'headers') && Reflect.has(error.headers, 'retry-after') && retryAfterStatusCodes.has(error.statusCode)) {
218+
let after = Number(error.headers['retry-after']);
219+
if (is.nan(after)) {
220+
after = Date.parse(error.headers['retry-after']) - Date.now();
221+
} else {
222+
after *= 1000;
231223
}
232224

233-
if (error.statusCode === 413) {
225+
if (after > options.retry.maxRetryAfter) {
234226
return 0;
235227
}
228+
229+
return after;
230+
}
231+
232+
if (error.statusCode === 413) {
233+
return 0;
236234
}
237235

238236
const noise = Math.random() * 100;

0 commit comments

Comments
 (0)
Please sign in to comment.