Error.toJSON() should exclude TLSSocket

[benoit-messager]

Describe the bug

When calling JSON.stringify(AxiosError.toJSON()) there is a circular reference error :

[TypeError: Converting circular structure to JSON
        --> starting at object with constructor 'TLSSocket'
        |     property 'parser' -> object with constructor 'HTTPParser'
        --- property 'socket' closes the circle]

The error does not occurs on all platforms.

To Reproduce

The code building the axios instance is spread amongst several classes with multiple conditions, i will try my best to summarize the configuration while keeping the code readable :

const axiosInstance = axios.create({
            httpsAgent:
                new Agent({
                    ca: this.certificate,
                }),
        });

  axiosInstance.interceptors.request.use((config: any) => {
            config.headers[correlationIdKey] = getCorrelationId();
            config.metadata = config.meta || {};
            config.metadata = { startTime: new Date() };
            return config;
        });

  axiosInstance.interceptors.response.use(
            (response: any) => {
                this.logResponse(partnerName, response, maskedFields?.responseFields);
                return response;
            },
            (error: any) => {
                this.logResponse(partnerName, error, maskedFields?.responseFields);
                return Promise.reject(error);
            }
        );

axiosInstance.get<unknown>(url)
            .catch((error: AxiosError) => {
                this.logError("getCustomer", error);  //This method use JSON.stringify(error.toJSON()) and throw the circular reference error
                throw new ApiError("NAG_4400", error.message);
            })

Code snippet

No response

Expected behavior

Calling JSON.stringify() with an axios error does not result in a circular reference error.

Axios Version

1.1.3

Adapter Version

No response

Browser

No response

Browser Version

No response

Node.js Version

v16.15.1

OS

OSX 12.6

Additional Library Versions

No response

Additional context/Screenshots

No response

[benoit-messager]

Just realize this is a duplicate of #4486

[deenairn]

This is definitely still happening in our environment, though our call stack is not the same. I've documented this at #5332, so I think this is still an active problem in some scenarios and a fix should be considered.