Skip to content

Commit 24787b1

Browse files
authored
feat: error event now includes response headers (#35)
2 parents aa51028 + cd6a69c commit 24787b1

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

lib/eventsource.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ function EventSource (url, eventSourceInitDict) {
221221
if (res.statusCode === 301 || res.statusCode === 307) {
222222
if (!res.headers.location) {
223223
// Server sent redirect response without Location header.
224-
failOnce({ status: res.statusCode, message: res.statusMessage })
224+
failOnce({ status: res.statusCode, headers: res.headers, message: res.statusMessage })
225225
return
226226
}
227227
if (res.statusCode === 307) reconnectUrl = url
@@ -232,7 +232,7 @@ function EventSource (url, eventSourceInitDict) {
232232

233233
// Handle HTTP errors
234234
if (res.statusCode !== 200) {
235-
failOnce({ status: res.statusCode, message: res.statusMessage })
235+
failOnce({ status: res.statusCode, headers: res.headers, message: res.statusMessage })
236236
return
237237
}
238238

test/eventsource_test.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -540,6 +540,7 @@ describe('HTTP Request', () => {
540540
const errors = startErrorQueue(es)
541541
const err = await errors.take()
542542
assert.equal(err.status, status)
543+
assert.notEqual(err.headers, undefined)
543544
})
544545
})
545546
})

0 commit comments

Comments
 (0)