Skip to content

Commit 45dad1a

Browse files
authored
fix: #54 add response status in error details (#58)
1 parent 570ae40 commit 45dad1a

File tree

1 file changed

+10
-16
lines changed

1 file changed

+10
-16
lines changed

src/index.js

+10-16
Original file line numberDiff line numberDiff line change
@@ -324,7 +324,13 @@ class AEMHeadless {
324324
messageValues: error.message
325325
})
326326
}
327+
327328
let apiError
329+
const sdkDetails = {
330+
serviceURL: this.serviceURL,
331+
endpoint,
332+
status: response.status
333+
}
328334
// 2. Handle Response error
329335
if (!response.ok) {
330336
try {
@@ -333,10 +339,7 @@ class AEMHeadless {
333339
} catch (error) {
334340
// 2.3 Response error: Couldn't parse JSON - no error defined in API response
335341
throw new RESPONSE_ERROR({
336-
sdkDetails: {
337-
serviceURL: this.serviceURL,
338-
endpoint
339-
},
342+
sdkDetails,
340343
messageValues: error.message
341344
})
342345
}
@@ -345,10 +348,7 @@ class AEMHeadless {
345348
if (apiError) {
346349
// 2.2 Response error: JSON parsed - valid error defined in API response
347350
throw new API_ERROR({
348-
sdkDetails: {
349-
serviceURL: this.serviceURL,
350-
endpoint
351-
},
351+
sdkDetails,
352352
messageValues: apiError
353353
})
354354
}
@@ -359,20 +359,14 @@ class AEMHeadless {
359359
} catch (error) {
360360
// 3.2. Response ok: Data error - Couldn't parse the JSON from OK response
361361
throw new RESPONSE_ERROR({
362-
sdkDetails: {
363-
serviceURL: this.serviceURL,
364-
endpoint
365-
},
362+
sdkDetails,
366363
messageValues: error.message
367364
})
368365
}
369366
// 3.2. Response ok: containing errors info
370367
if (data && data.errors) {
371368
throw new RESPONSE_ERROR({
372-
sdkDetails: {
373-
serviceURL: this.serviceURL,
374-
endpoint
375-
},
369+
sdkDetails,
376370
messageValues: data.errors.map((error) => error.message).join('. ')
377371
})
378372
}

0 commit comments

Comments
 (0)