Adding HTTP status code to error.toJSON

[byrne-greg]

Issue Summary

Previously it was reported in #2947 that the Axios error.toJSON() was not outputting the HTTP status code.

For example;

const axios = require('axios');

axios.get('/404')
  .catch(function (error) {
    console.log(error.toJSON());
  })

would result in the following error message that did not have the HTTP status code of 404 instead only containing the code in the nested in the error message string (such as "Request failed with status code 404").

The Fix

With the following addition to the toJSON function's return object in enhanceError.js:

status: this.response && this.response.status ? this.response.status : null

the response HTTP status code will be added when a response is available. If no response is available (i.e. we can't derive a HTTP status code), we will use null as the value to imply it is intentionally unavailable.

Failing Test without addition

createError.spec.js (existing test-class) (test-amendment)

npx jasmine **/*/createError.spec.js
npx: installed 13 in 1.863s
Randomized with seed 49409
Started
F.

Failures:
1) core::createError should create an Error that can be serialized to JSON
  Message:
    Expected undefined to be 200.
  Stack:
    Error: Expected undefined to be 200.
        at <Jasmine>
        at UserContext.<anonymous> (/Users/gregbyrne/Personal_Storage/Personal/axios-fork/test/specs/core/createError.spec.js:26:25)
        at <Jasmine>

enhanceError.spec.js (existing test-class) (new test)

npx jasmine **/*/enhanceError.spec.js
npx: installed 13 in 1.481s
Randomized with seed 46485
Started
..F

Failures:
1) core::enhanceError should serialize to JSON with a status of null when there is no response
  Message:
    Expected undefined to equal null.
  Stack:
    Error: Expected undefined to equal null.
        at <Jasmine>
        at UserContext.<anonymous> (/Users/gregbyrne/Personal_Storage/Personal/axios-fork/test/specs/core/enhanceError.spec.js:24:32)
        at <Jasmine>

3 specs, 1 failure
Finished in 0.01 seconds

Passing Test with addition

Execution specifically on the *Error.spec.js tests

npx jasmine **/*/*Error.spec.js 
npx: installed 13 in 1.883s
Randomized with seed 07705
Started
.....
5 specs, 0 failures
Finished in 0.012 seconds

Execution with npm test

Chrome 81.0.4044 (Mac OS X 10.15.4): Executed 211 of 211 SUCCESS (3.457 secs / 3.403 secs)
....................................................
Firefox 75.0.0 (Mac OS X 10.15.0): Executed 211 of 211 SUCCESS (2.179 secs / 2.146 secs)

Side Note: Cannot seem to run these in Safari. When it launches the karma runner, it opens the Safari browser with a Choose file modal already focused to a redirect.html. Also it hangs running on Opera which I don't believe I have installed. When I sigkill, testing moves on to the the bundlesize test

Internal client.html addition

Additionally, I've added a display to the internal client.html which is started using npm start.

This now displays the result of an error in the main display (under an Error div) that uses toJSON() as well as displaying the error object in the console.

No Error

On startup

Successful response / no error

Error

With a 404 error

[byrne-greg]

After submitting this PR did I realise that there is 123 (at time of writing) open PR's on this project awaiting integration.

Is it safe to say this one likely won't be merged in anytime soon?

[jasonsaayman]

Hi,

I will review soon. We are trying to release a v0.20 as soon as possible.

[MauricioLoya]

Good job!

[jasonsaayman]

Looks good perfect for v0.20.0

[github-actions]

Hello! 👋 \n\nThis pull request is being automatically marked as stale because it has not been updated in a while. Please confirm that the issue is still present and reproducible. If no updates or new comments are received the pull request will be closed in a few days. \n\nThanks

[atlanteh]

Can this please be merged? We have hacks in multiple places to fix this. thx :)

[byrne-greg]

@emilyemorehouse @jasonsaayman
do you know if this is on the roadmap for release? it was previously considered for v0.20