Skip to content

Commit e7e72cd

Browse files
committed
fixup
1 parent 5426629 commit e7e72cd

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

lib/handler/retry-handler.js

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ class RetryHandler {
136136
setTimeout(() => cb(null), retryTimeout).unref()
137137
}
138138

139-
onResponseStart (controller, statusCode, statusMessage, headers) {
139+
onResponseStart (controller, statusCode, headers, statusMessage) {
140140
this.retryCount += 1
141141

142142
if (statusCode >= 300) {
@@ -145,8 +145,8 @@ class RetryHandler {
145145
this.handler.onResponseStart?.(
146146
controller,
147147
statusCode,
148-
statusMessage,
149-
headers
148+
headers,
149+
statusMessage
150150
)
151151
return
152152
} else {
@@ -207,8 +207,8 @@ class RetryHandler {
207207
this.handler.onResponseStart?.(
208208
controller,
209209
statusCode,
210-
statusMessage,
211-
headers
210+
headers,
211+
statusMessage
212212
)
213213
return
214214
}
@@ -254,8 +254,8 @@ class RetryHandler {
254254
this.handler.onResponseStart?.(
255255
controller,
256256
statusCode,
257-
statusMessage,
258-
headers
257+
headers,
258+
statusMessage
259259
)
260260
} else {
261261
throw new RequestRetryError('Request failed', statusCode, {
@@ -277,9 +277,7 @@ class RetryHandler {
277277
}
278278

279279
onResponseError (controller, err) {
280-
assert(controller)
281-
282-
if (controller?.aborted || isDisturbed(this.opts.body)) {
280+
if (!controller || controller.aborted || isDisturbed(this.opts.body)) {
283281
this.handler.onResponseError?.(controller, err)
284282
return
285283
}

0 commit comments

Comments
 (0)