Skip to content

Commit 838ff77

Browse files
committed
Revises Container Service
This commit revises the container service to use a seperate instance of axios when making container requests. This prevents an inheritence issue causing incorrect error codes to be thrown. Signed-off-by: Lui de la Parra <Lui@mutesymphony.com>
1 parent 0563b1d commit 838ff77

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

src/services/container.service.js

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,19 @@
11
'use strict';
22

3+
const axios = require('axios');
4+
const axiosCookieJarSupport = require('axios-cookiejar-support').default;
35
const fs = require('fs');
46
const path = require('path');
57
const toArray = require('stream-to-array');
68
const { CookieJar } = require('tough-cookie');
79
const mime = require('mime-types');
810
const uuidv4 = require('uuid/v4');
911
const _ = require('lodash');
10-
const { instance } = require('./request.service');
12+
const { interceptResponse } = require('./request.service')
13+
14+
const instance = axios.create();
15+
16+
axiosCookieJarSupport(instance);
1117

1218
/**
1319
* @class Container Service
@@ -19,7 +25,7 @@ const { instance } = require('./request.service');
1925
* @param {Any} error We publishing error.
2026
*/
2127
instance.interceptors.response.use(
22-
response => response,
28+
response => interceptResponse(response),
2329
error => handleError(error)
2430
);
2531

src/services/request.service.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,5 +77,5 @@ instance.interceptors.response.use(
7777
);
7878

7979
module.exports = {
80-
instance
80+
instance, interceptResponse
8181
};

0 commit comments

Comments
 (0)